Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Todo: Als Tab in yrewrite umsetzen #1

Closed
skerbis opened this issue Oct 9, 2017 · 14 comments
Closed

Todo: Als Tab in yrewrite umsetzen #1

skerbis opened this issue Oct 9, 2017 · 14 comments
Labels

Comments

@skerbis
Copy link
Member

skerbis commented Oct 9, 2017

No description provided.

@tbaddade
Copy link
Member

tbaddade commented Oct 9, 2017

Als Tab in yrewrite umsetzen

Finde ich nicht gut. Nur AddOns sollten ihre eigenen PlugIns mitbringen. "Installer Updates"

@gharlan
Copy link
Member

gharlan commented Oct 9, 2017

Finde ich nicht gut. Nur AddOns sollten ihre eigenen PlugIns mitbringen. "Installer Updates"

Wie schon an anderer Stelle angemerkt, bin ich hier klar anderer Meinung. Ich finde es legitim, wenn sich Addons als Subpage bei anderen Addons einbinden, und deswegen habe ich es in R5 auch so gebaut, dass es sehr einfach möglich ist (wenn das Mutteraddon die Subpages "korrekt" einbindet).

Ich finde, dieses Addon ist ein Paradebeispiel dafür, wo es Sinn ergibt. Sehe da null Vorteile drin (im Gegenteil), wenn das Addon stattdessen eine eigene Haupt-Page erstellen würde.

@skerbis
Copy link
Member Author

skerbis commented Oct 9, 2017

Also, ein anderes Addon sollte kein Tab in einem anderen AddOn haben? Auch nicht wenn es darauf aufbaut? Ich empfinde es eigentlich als sinnlos für jede kleine Einstellung einen Hauptmenüpunkt zu verbrauchen. Das hier meinte ich
bildschirmfoto 2017-10-09 um 20 27 30

@skerbis
Copy link
Member Author

skerbis commented Oct 9, 2017

Ich bitte um Klärung im Core-Team. Wäre echt schade, wenn Ihr was dagegen hättet.

@gharlan
Copy link
Member

gharlan commented Oct 9, 2017

Wie schon an anderer Stelle angemerkt

Hier meinte ich diese Diskussion: redaxo/redaxo#1042

@tbaddade
Copy link
Member

tbaddade commented Oct 9, 2017

Sehe da null Vorteile drin (im Gegenteil), wenn das Addon stattdessen eine eigene Haupt-Page erstellen würde.

Verstehe gar nicht warum als Extra-AddOn. Warum nicht als PR im YRewrite Addon anbieten?

@skerbis
Copy link
Member Author

skerbis commented Oct 9, 2017

Ich sehe es aktuell noch nicht für gut genug an. Wenn #3 umgesetzt ist vielleicht. So grenzt man evtl. aber individuelle Wünsche ein.

@gharlan
Copy link
Member

gharlan commented Oct 9, 2017

Verstehe gar nicht warum als Extra-AddOn. Warum nicht als PR im YRewrite Addon anbieten?

Das ist eine Frage, die sich ja immer wieder stellt, und wie ich finde, nicht so leicht zu beantworten ist.

Da müsste ja erstmal auch geklärt werden, ob wir das überhaupt direkt in YRewrite drin haben wollen.
Über das Theme habe ich mit Jan im Laufe der Jahre immer mal wieder diskutiert, ob wir YRewrite eher so clean wie möglich halten wollen, und es somit für speziellere Wünsche bei der programmatischen Lösung (eigene Scheme-Klasse) bleiben soll, oder ob wir eher weitere Optionen hinzufügen wollen, sodass möglichst viele sich ihre individuellen Vorstellungen für das Schema zusammen klicken können. Und wenn ja, wie weit man da geht.
So richtig entschieden waren wir da nie.

Aber abseits dieser Frage, gibt es auch so Vor- und Nachteile.
Nachteil der Variante "eigenes Addon" ist klar: Ein weiteres Addon. Was man so mitkriegt, stört das schon einige, wie viele Addons man inzwischen meist so "braucht".
Vorteile allerdings gibt es auch: Man ist völlig frei von den Yakamara-Entscheidungen, die FOR-Community könnte hier eigenständig diverse Optionen sammeln, die uns in YRewrite vielleicht eher zu viel wären, oder zu instabil, oder sonstwas.
YRewrite würde so "clean" bleiben, und wer sich gerne individueller das Schema zusammen klicken möchte, müsste sich halt das Zusatzaddon installieren.

Ich bin hier offen für beide Wege, kann beiden was abgewinnen.

@skerbis
Copy link
Member Author

skerbis commented Oct 9, 2017

Ich bin auch eher für die Clean-Variante. Wer die URL-Schemes aus diesem AddOn nicht haben will, kann eine andere Lösung verwenden. (aktuell ist mir eine weitere bekannt). Aber es gibt eben Sonderfälle die ein Konfigurator oder Selektor nicht abdeckt. Es sollte offen bleiben. Ich möchte nun nur geklärt wissen ob ich mich mit einem Tab einklinken kann/darf oder nicht. Sonst muss grundsätzlich über ein Bereich nachgedacht werden wo kleine Settings untergebracht werden.

@skerbis
Copy link
Member Author

skerbis commented Oct 9, 2017

@tbaddade @gharlan Gegenvorschlag um es sichtbar als externes AddOn zu machen:
bildschirmfoto 2017-10-09 um 21 26 43

@skerbis
Copy link
Member Author

skerbis commented Oct 9, 2017

In redaxo/redaxo#1042 hatte ich auch meine Bedenken, aber irgendwie leuchtet es mir hier auch nicht mehr ein, einen eigenen Menüpunkt dafür zu haben zu müssen.

@gharlan
Copy link
Member

gharlan commented Oct 9, 2017

Ich möchte nun nur geklärt wissen ob ich mich mit einem Tab einklinken kann/darf oder nicht.

Es gibt im Core-Team da Dissens, der sich wohl nicht auflösen lässt.
Aber so oder so: Es ist möglich, und keiner kann es verbieten. Es ist somit eine FOR-Entscheidung, ob das Addon sich als Subpage einhaken soll. Ich stimme klar dafür.
Wie das nun genau gestaltet ist (Screenshot 1 oder 2) ist mir egal.

@skerbis
Copy link
Member Author

skerbis commented Oct 9, 2017

ok. Danke für's Feedback

@skerbis skerbis closed this as completed Oct 9, 2017
@schuer
Copy link
Member

schuer commented Oct 10, 2017

Was Gregor sagt. Und Screenshot 1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants