phpBB.de

LukeWCS hat geschrieben: 18.11.2024 19:51 Was nehmen wir? Optionsliste oder Pulldown?

1. Das Twig Makro erlaubt es jetzt auch Multiselect Menüs erzeugen zu können. Man kann also mit demselben Makro sowohl einfache Auswahlmenüs als auch Mehrfachauswahlmenüs generieren. Bei letzterem werden einfach 2 weitere Parameter übergeben, der erste ist ein Bool zum aktivieren von Multiselect, der zweite definiert, wieviele Elemente standardmässig direkt angezeigt werden sollen.
2. Bei Select ist es nicht länger notwendig, zusätzlich zur Elementliste auch die Config Variable mit der zuletzt ausgewählten Position zu übergeben. Das läuft jetzt alles über ein einziges Objekt. Im jeweiligen Controller kümmert sich eine neue PHP Funktion um die Generierung des Template Objekts für das Twig Makro.

{{ common.select('user_rtng_location', RTNG_LOCATION, RTNG_LOCATION_OPTIONS) }}

{{ common.select('user_rtng_location', RTNG_LOCATION_OPTIONS) }}

			'RTNG_LOCATION'			=> $user_data['user_rtng_location'],
			'RTNG_LOCATION_OPTIONS' => [
				'RTNG_TOP'	 	=> 'RTNG_TOP',
				'RTNG_BOTTOM'	=> 'RTNG_BOTTOM',
				'RTNG_SIDE'	 	=> 'RTNG_SIDE',
				'RTNG_SEPARATE' => 'RTNG_SEPARATE',
			],

			'RTNG_LOCATION_OPTIONS' => $this->select_struct($user_data['user_rtng_location'], [
				'RTNG_TOP'	 	=> 'RTNG_TOP',
				'RTNG_BOTTOM'	=> 'RTNG_BOTTOM',
				'RTNG_SIDE'	 	=> 'RTNG_SIDE',
				'RTNG_SEPARATE' => 'RTNG_SEPARATE',
			]),

LukeWCS hat geschrieben: 18.11.2024 20:58 Ich denke du verwechselst das jetzt mit einem Mehrfachauswahl Pulldown (select + multiselect), mit Optionsliste meinte ich jedoch Radio Buttons für 2+ Optionen wo nur eine davon ausgewählt werden kann.

Okay, nehmem wir das einfache Pulldown (select).

Das Twig Makro erlaubt es jetzt auch Multiselect Menüs erzeugen zu können. .... Im jeweiligen Controller kümmert sich eine neue PHP Funktion um die Generierung des Template Objekts für das Twig Makro.

IMC hat geschrieben: 18.11.2024 23:07 War für mich bis heute immer nur eine "Radio button group"

Als "Value" aber keinen String, wie in RTNG_LOCATION_OPTIONS, sondern binör mit 1 und 0. Dann kann der restliche Code fortbestehen.

1. Neben dem Fix von Udo hab ich noch einen weiteren vorgenommen; im ACP Modul sind beim Umbau die Lokalisierungen für Bekanntmachungen und globale Bekanntmachungen verlorengegangen.
2. Du übersteuerst ToggleCtrl im UCP und Benutzerverwaltung unterschiedlich. Im UCP überschreibst du die globale Variable von TC, das ist in dem Fall kein Problem, da Template Variablen ja Kontext-abhängig generiert werden und im UCP eh nur Radios benutzt werden sollten. Aber deine Übersteuerung in der Benutzerverwaltung gefällt mir besser, die sollten wir konsequent nutzen und damit wäre auch eine potentielle Kollisionsgefahr sofort vom Tisch.
3. Bei der einfachen Anzeige nutzt du die Beschreibung des zweiten Schalters dazu, einen generierten Link darzustellen, damit der Admin gleich weiss wie das erreicht werden kann. Du hast das ans Ende vom zweiten Schalter gesetzt, vermutlich damit die beiden Schalter kompakt beieinander stehen. Allerdings ist die Position des Links im Template hardcodet, ich habe daraus eine Platzhalter-Variable gemacht, damit Übersetzer das frei positionieren können. Aber inzwischen hätte ich hier einen ganz anderen Vorschlag: daraus eine reguläre Element-Liste machen, wie alle anderen Einstellungen auch. Dabei die jetzige Beschreibung als Label nutzen und dann den Link nach rechts packen und direkt anklickbar gestalten. Da könnte man dann ausnahmsweise mit target="_blank" arbeiten, damit bei Admins keine geänderten Einstellungen verlorengehen können. Den Link natürlich wie jetzt komplett ausschreiben, denn man soll ja direkt sehen, wohin es geht. So hätte man a) alle Seiten von RT direkt verlinkt und b) hätte man gleich eine prima "Überschrift" für die beiden Schalter und der Abschnitt mit den beiden Schaltern und dem Link würde "geordneter" aussehen.
4. Für die PHP Funktion select_struct() hatte ich zuerst rtng_functions.php im Visier, aber da gehts komplett um die Darstellung von RT. Darum habe ich eine neue Klasse im Ordner controller angelegt und als Dienst definiert, damit man da allgemeine Funktionen für die verschiedenen Controller bei RT einbauen kann.

{%- set switch_type = TOGGLECTRL_TYPE ?? 'toggle' %}
{%- set switch_type = TOGGLECTRL_RTNG ? TOGGLECTRL_RTNG : switch_type %}

{%- set switch_type = TOGGLECTRL_RTNG ?? TOGGLECTRL_TYPE ?? 'toggle' %}

{%- set switch_type = TOGGLECTRL_RTNG ?: switch_type %}

{%- set switch_type = TOGGLECTRL_RTNG ? TOGGLECTRL_RTNG : switch_type %}

LukeWCS hat geschrieben: 20.11.2024 17:10 im ACP Modul sind beim Umbau die Lokalisierungen für Bekanntmachungen und globale Bekanntmachungen verlorengegangen.

Aber inzwischen hätte ich hier einen ganz anderen Vorschlag: daraus eine reguläre Element-Liste machen, wie alle anderen Einstellungen auch. Dabei die jetzige Beschreibung als Label nutzen und dann den Link nach rechts packen und direkt anklickbar gestalten. ... So hätte man a) alle Seiten von RT direkt verlinkt und b) hätte man gleich eine prima "Überschrift" für die beiden Schalter und der Abschnitt mit den beiden Schaltern und dem Link würde "geordneter" aussehen.

Da könnte man dann ausnahmsweise mit target="_blank" arbeiten,

Für die PHP Funktion select_struct() hatte ich zuerst rtng_functions.php im Visier, aber da gehts komplett um die Darstellung von RT. Darum habe ich eine neue Klasse im Ordner controller angelegt und als Dienst definiert, damit man da allgemeine Funktionen für die verschiedenen Controller bei RT einbauen kann.

Das könnte man, sofern wir die Übersteuerung konsequent nutzen, kompakter gestalten:

Code: Alles auswählen

{%- set switch_type = TOGGLECTRL_RTNG ?? TOGGLECTRL_TYPE ?? 'toggle' %}

IMC hat geschrieben: 20.11.2024 22:43 select_struct() kann ich gerade nicht zuordnen. Werde ich aber sehen wenn Alpha3 online ist.

In der neuen Klasse sollen dann Helper-Funktionen zusammen gefasst werden?

Irgendwann wird der Nullish coalescing operator auch in mein "Fleisch und Blut" übergehen.

LukeWCS hat geschrieben: 20.11.2024 23:14 hab ich den Bug-Jagd-Modus aktiviert und paar Debugs und Versuche später gings dem Bug auch schon an den Kragen. Ein schnöder Tippfehler bei einer Konfig Variable. ^^

IMC hat geschrieben: 20.11.2024 23:50 Eine meiner unliebsten Tätigkeiten.