lms icon indicating copy to clipboard operation
lms copied to clipboard

enhancement: variable phpui.instance_label (default null, type: text)…

Open interduo opened this issue 4 years ago • 14 comments

Udostępniłem instancję testową moim użytkownikom. Obecnie wizualnie instancje wyglądają identycznie. Proponuję je odróżnić kilkoma szczegółami przy logowaniu i normalnej pracy.

interduo avatar Nov 08 '20 17:11 interduo

A nie możesz po prostu przez wpis w DNS to zrobić? Dziwne to jak dla mnie.

chilek avatar Nov 08 '20 17:11 chilek

Czy mówisz o tym żeby stworzyć: phpui.show_instance_name i jeśli ustawiona na true to pokazywać {$smarty.server.SERVER_NAME} w tych miejscach co zaznaczyłem ?

interduo avatar Nov 08 '20 17:11 interduo

Jak naciśniesz F11 to niewidać różnicy między instancjami.

interduo avatar Nov 08 '20 17:11 interduo

Sam sobie już odpowiedziałeś ;-)

chilek avatar Nov 08 '20 17:11 chilek

Wolę się upewnić czasami.

A co jak ktoś ma dwie instancje na jednym hostname (z oddzielnymi bazami)? Może domyślnie prezentować w tych miejscach hostname a jeśli jest phpui.instance_label to etykietkę?

interduo avatar Nov 08 '20 17:11 interduo

instance_label nadpisuje hostname, myślę, że to jest: wystarczające, elastyczne i minimalistyczne

interduo avatar Nov 08 '20 18:11 interduo

To jak już to robisz to może warto z instance_name zrobić coś takiego, że mogłoby być również ciągiem formatującym prezentowaną nazwę - np. %hostname, %dbname, itp.

chilek avatar Nov 08 '20 18:11 chilek

Czyli:

  • dorobić obsługę symboli specjalnych w tej zmiennej
  • w domyślną wartość tej zmiennej wrzucić %hostname ?

interduo avatar Nov 08 '20 18:11 interduo

Skoro już robisz rzecz nieistotną to przynajmniej zrób od razu porządnie :D

chilek avatar Nov 08 '20 19:11 chilek

Co znaczy porządnie?;) Wiesz, że każda rzecz można zrobić na wiele sposobów?

interduo avatar Nov 08 '20 20:11 interduo

Co znaczy porządnie?;)

No tak - to po dawien dawnia widać, że nie wiesz co to znaczy :D

chilek avatar Nov 08 '20 21:11 chilek

Z innej beczki: wiesz, że instancję testową możesz również oznaczyć wskazując własny plik z arkuszem styli przez: https://github.com/chilek/lms-plus/wiki/Zmienne-konfiguracyjne-LMS-Plus-phpui#phpuistyle

chilek avatar Nov 09 '20 09:11 chilek

To co piszesz przyda się gdy będzie darkmode support ;-)

interduo avatar Nov 09 '20 09:11 interduo

Czy marzy Ci się coś jeszcze w ramach tego PR?

interduo avatar Nov 09 '20 15:11 interduo