search

TN03 / expandcontract

http://svasti.de/en/?Welcome/Expandcontract
0 stars 1 forks source link

Parameter-Namen vereinheitlichen #19

Closed frase-git closed 2 years ago

frase-git commented 2 years ago

Aktuell heißen die (einige) Parameter: show-close, max-height, show-inline, content-padding, auto-close aber: firstopen

Das sollte irgendwie vereinheitlicht werden. Entweder alle mit Bindestrich: first-open oder alle ohne.

Ich neige dazu, alle ohne Bindestrich zu schreiben - das tippt sich schneller. Mit Bindestrich ist es allerdings etwas übersichtlicher.

TN03 commented 2 years ago

Wen wir hier, bzw. davor https://github.com/TN03/expandcontract/blob/ec4354e27e6d07fd7b37cefa10e00e9912ec49aa/index.php#L44 noch mittels str_replace() alle Bindestriche heraus filtern (und die paar Parameter dann im folgenden Code anpassen) kann man die Parameter mit / ohne Bindestrich und gemischt in Groß- / Kleinschreibung übergeben. Wäre vielleicht auch eine Lösung?

olape-git commented 2 years ago

Ich wäre dann eher für first-open, es ist im Ganzen irgendwie besser zu lesen. Wir können Holgers Idee doch aber umsetzen, man kann schon jetzt recht wild mit den Parametern umgehen, dann können wir die Wahl ob mit oder ohne Bindestrich auch noch dem Nutzer überlassen.

frase-git commented 2 years ago

Wir können Holgers Idee doch aber umsetzen, man kann schon jetzt recht wild mit den Parametern umgehen, dann können wir die Wahl ob mit oder ohne Bindestrich auch noch dem Nutzer überlassen.

Tippfehler-tolerante Software die trotzdem richtig arbeitet - das kann nur gut sein! ;-) In der Plugin-Beschreibung würde ich dann aber alle nur mit BIndestrich erwähnen - das sieht in jedem Fall übersichtlicher aus.

TN03 commented 2 years ago

Closed with https://github.com/TN03/expandcontract/pull/23