Languagemenü - Flaggen oder Kürzel

[lck]

Dort gibts du der Sache einen Titel (Summary) und trägst etwas Beschreibungstext (Description) ein.
Nach dem Absenden erscheinen die Änderungen in deinem Repo.

Ist online https://github.com/lck-git/cmsimple-xh. Jetzt sollte das erstmal getestet werden, bevor ich einen PR mache. Da wird auch noch einiges an Änderungen kommen.

Im core.css habe ich es auch beispielhaft gestylt. Den Teil könnt ihr auch mal löschen damit ihr seht, wie es ohne Style aussieht. Außerdem wird das Sprachmenü nun mit einen <div class="xh_langMenu">...</div> umgeben.

Probleme konnte ich auch schon feststellen, wenn eine andere Konfig-Option als "flags" (=Standard-Einstellung) verwendet wird (Template JekyllAndHyde, Split-it). Bei "overlay_03" bekomme ich sogar eine weiße Seite mit Debug-Meldung. Dieses Template verwendet bereits eine angepasste function languagemenu() in einer gesonderten PHP-Datei, da beißt sich wohl etwas.
Bei "Little Jo" ist die Einstellung in der XH-Konfiguration wirkungslos, da es mit einem Template-Plugin ausgeliefert wird und die Einstellung für das Sprachmenü hier vorzunehmen sind und auch funktionieren. Da beißt sich anscheinend nichts .

[olape]

Ohne das jetzt getestet zu haben.

Code: Select all

 if (isset($cf['language']['menu'])

Könnte wahrscheinlich sogar ganz wegfallen.
Da es aus der config kommt, kann es eigentlich nicht sein, dass es nicht gesetzt ist.
Aber wenigstens könnte man es auf eine einmalige Abfrage eindampfen und entsprechend alle weiteren eine zusätzliche „Ebene“ einrücken.
Also statt jedesmal if isset

Code: Select all

 if (isset($cf['language']['menu'])
         && $cf['language']['menu'] == 'flags'
         …
         ) {

Code: Select all

 if (isset($cf['language']['menu']) {
                   if ($cf['language']['menu'] == 'flags') {
                            …
                   } elseif ($cf['language']['menu'] == 'shortname') {
                            …
                   }
         

Einfach um es kürzer und damit übersichtlicher zu halten.

[frase]

Jetzt sollte das erstmal getestet werden, bevor ich einen PR mache.

Richtig so.

Probleme konnte ich auch schon feststellen, wenn eine andere Konfig-Option als "flags" (=Standard-Einstellung) verwendet wird (Template JekyllAndHyde, Split-it). Bei "overlay_03" bekomme ich sogar eine weiße Seite mit Debug-Meldung. Dieses Template verwendet bereits eine angepasste function languagemenu() in einer gesonderten PHP-Datei, da beißt sich wohl etwas.

Also eine weiße Seite habe ich mit den genannten Templates nicht hinbekommen, aber kleinere Probleme sind offensichtlich.
Vielleicht sollte man im Auswahlfeld eine Auswahlmöglichkeit haben "default" - wo dann überhaupt nichts anderes passiert als bisher?
Also alles Neue nur bei den anderen Optionen?
So könnten bisherige Templates weiter bestehen. Ich habe nämlich auch einige Dinger in freier Wildbahn laufen, bei denen das Sprachmenü manipuliert ist. Mir graut schon davor, wenn ich das mal teste.

Im core.css habe ich es auch beispielhaft gestylt. Den Teil könnt ihr auch mal löschen damit ihr seht, wie es ohne Style aussieht.

Also ich bin mit den Tests noch nicht durch - aber, das dürfte bestimmt nicht so bleiben. Orange Rahmen um das dd-Menü? Und runde Ecken? Sowas dürfte nicht als Standard drinstehen. Und dass es immer nach unten ausklappt ist auch nicht in jedem Fall günstig (split-it: Menü unten).
Na ja, das braucht alles Zeit und viele Tests. Uns treibt ja niemand.

Einen konkreten Vorschlag hätte ich aber jetzt schon:
Statt nur "Sprache" würde ich "Sprachauswahl" ("Select Language") verwenden. Das scheint mir irgendwie eindeutiger.

Aber wenigstens könnte man es auf eine einmalige Abfrage eindampfen und entsprechend alle weiteren eine zusätzliche „Ebene“ einrücken.
Also statt jedesmal if isset
...
Einfach um es kürzer und damit übersichtlicher zu halten.

Mal so als PHP-Laie gefragt:
Wäre da nicht etwas mit "switch" und "case" angebrachter?

[olape]

Wäre da nicht etwas mit "switch" und "case" angebrachter?

Das Ergebnis wäre das gleiche.
Aber ja, könnte man auch damit realisieren.
Würde es vielleicht noch übersichtlicher machen.

[lck]

Code: Select all

 if (isset($cf['language']['menu']) {
                   if ($cf['language']['menu'] == 'flags') {
                            …
                   } elseif ($cf['language']['menu'] == 'shortname') {
                            …
                   }
         

Einfach um es kürzer und damit übersichtlicher zu halten.

Danke, erledigt.

Gleich eine Frage zu Github Desktop. Wenn ich jetzt Änderungen gemacht habe, muss ich das dann genauso hochladen wie von Frank beschrieben oder anders?

[frase]

Gleich eine Frage zu Github Desktop. Wenn ich jetzt Änderungen gemacht habe, muss ich das dann genauso hochladen wie von Frank beschrieben oder anders?

Frank sagt: Genau so.
Änderungen lokal ablegen.
Im Desktop unter Changes committen.

[lck]

Also eine weiße Seite habe ich mit den genannten Templates nicht hinbekommen, aber kleinere Probleme sind offensichtlich.

Komisch, bei overlay_03 kam

Cannot redeclare languagemenu_custom() (previously declared in D:\xampp\htdocs\Portable_XH-3.0alpha1\www\cmsimple-xh\templates\overlay_03\langmenu_custom.php:7) in D:\xampp\htdocs\Portable_XH-3.0alpha1\www\cmsimple-xh\templates\overlay_03\langmenu_custom.php on line 7

Wahrscheinlich ein Problem nur in Portable_XH. Muss ich mal unter Xampp testen.

Vielleicht sollte man im Auswahlfeld eine Auswahlmöglichkeit haben "default" - wo dann überhaupt nichts anderes passiert als bisher?
Also alles Neue nur bei den anderen Optionen?
So könnten bisherige Templates weiter bestehen. Ich habe nämlich auch einige Dinger in freier Wildbahn laufen, bei denen das Sprachmenü manipuliert ist. Mir graut schon davor, wenn ich das mal teste.

Die Option "flags" ist ja default. Ich habe jetzt das jetzt auch noch geändert zu "flags (default). Und wenn wir dann alle meine Teststyles im core.css entfernen, sollte alles bem Alten bleiben. Das war jetzt nur mal dazu gedacht um euch das Testen zu erleichtern.

Ich bin dafür quasi nix in der core.css zu ändern/einzufügen und das dem Template-Designer bzw. den User zu überlassen, wenn er das Sprachmenü anders als nur mit Flaggen haben will. Das umgebende div mit der Klasse xh_langMenu könnte man auch wieder entfernen. Andererseits sollte das keine Probleme verursachen.

Einen konkreten Vorschlag hätte ich aber jetzt schon:
Statt nur "Sprache" würde ich "Sprachauswahl" ("Select Language") verwenden. Das scheint mir irgendwie eindeutiger.

Bisher habe ich der Einfachheit halber das hier verwendet $tx['menu']['tab_language']. Müsste man einen neuen Text anlegen, das wäre auch kein Problem.

[lck]

Änderungen online https://github.com/lck-git/cmsimple-xh. (CSS und umgebendes div noch nicht geändert!)

Wie kann ich den Commit summary ändern?
Aus "Languagemenu modified #2" wurde online "Languagemenu modified cmsimple-xh#2" mit Verlinkung

[frase]

Wie kann ich den Commit summary ändern?
Aus "Languagemenu modified #2" wurde online "Languagemenu modified cmsimple-xh#2" mit Verlinkung

Ich fürchte, das geht nicht.

Änderungen online https://github.com/lck-git/cmsimple-xh. (CSS und umgebendes div noch nicht geändert!)

Da scheint etwas nicht zu stimmen.
Normale Flaggen werden nicht mehr angezeigt.

[frase]

Die Option "flags" ist ja default. Ich habe jetzt das jetzt auch noch geändert zu "flags (default). Und wenn wir dann alle meine Teststyles im core.css entfernen, sollte alles bem Alten bleiben. Das war jetzt nur mal dazu gedacht um euch das Testen zu erleichtern.

Ich bin dafür quasi nix in der core.css zu ändern/einzufügen und das dem Template-Designer bzw. den User zu überlassen, wenn er das Sprachmenü anders als nur mit Flaggen haben will. Das umgebende div mit der Klasse xh_langMenu könnte man auch wieder entfernen. Andererseits sollte das keine Probleme verursachen.

Ja, theoretisch sollte als Standard gar nichts gestylt sein.
Aber, diesen ungestylten Standard kann wohl kaum jemand so wie er ist verwenden. Das sieht schon sehr schräg aus.
Mir ist aber bisher noch nix passendes eingefallen