Skip to content
This repository has been archived by the owner on Jan 1, 2020. It is now read-only.

Addons mit eignenen Pages ermöglichen sich über EP PAGE_CONTENT_HEADER einzuhängen #94

Closed
ghost opened this issue Jan 29, 2013 · 28 comments
Labels

Comments

@ghost
Copy link

ghost commented Jan 29, 2013

Achtung! Es kommt ne Issue Lawine von mir :) (Gregor wollte das...;))

https://www.dropbox.com/s/kgekz48mhlcxvl0/rexseo_metainfos.png

https://www.dropbox.com/s/2hyeij5mtqecgbe/rex_multiupload_patches.png

auf be_search achten!
http://www.redaxo.org/de/forum/wunschliste-f5/wunsche-fur-die-nachste-r4-t18855.html#p103859

@jdlx
Copy link
Contributor

jdlx commented Feb 4, 2013

Die Idee mit den "tabs" find ich sehr gut.. was ich dabei aber sehr wünschenswert fände, wenn das nicht zwingend bedeuten müßte die Ausgabe eines solchen tabs wieder komplett zu Fuß stricken zu müssen - zumal wenn man einfach nur normale Metinfo Felder anzeigen will...

Idee: Vergleichbar dem rein strukturierenden legend typ der metainfos einen typ tab (oder wie auch immer) einführen, der Metainfo Felder - welche diesem tab per param zugeordnet sind - eben dort anzeigt.

@ghost
Copy link
Author

ghost commented Feb 4, 2013

zumal wenn man einfach nur normale Metinfo Felder anzeigen will...

das willst vielleicht du, aber andere wollen hier vielleicht auch noch js zeugs und so reinstricken. ich bräuchte hier in jedemfall volle flexibilität egal ob ich dann was "zu fuß" stricken muss.

und für metainfo felder gibts doch die meta info seite ;)

p.s.: wenn aber beides geht isses mir auch recht :)

@jdlx
Copy link
Contributor

jdlx commented Feb 4, 2013

Ich sprach nirgends davon, daß das eine das andere ersetzen soll.. es geht mir darum für den Fall das man nur Standard Metainfos in nem eigenen tab anzeigen will, nicht die Klimmzüge einer eigenen include page machen muß.

@jdlx
Copy link
Contributor

jdlx commented Feb 4, 2013

ps.: tabs Konzept für die Mediapool metas zumindest mal mit andenken.. die Problematik dort is faktisch dieselbe.

@jdlx
Copy link
Contributor

jdlx commented Feb 4, 2013

nochwas.. ;) ich find das nimmer wo ich das gemacht hatte, aber ich hab mal testweise n patch gemacht, wo die metainfos im editier modus schon geladen, aber ausgeblendet sind.. on click auf Metadaten/Sonstiges -> block wird eingeblendet. Hatte was sehr geschmeidiges..

@staabm
Copy link
Member

staabm commented Feb 4, 2013

Immer laden der Metainfos wäre mir zu viel Overhead. Mittels onclick via AJAX nachladen fände ich da eine bessere Variante.

@ghost
Copy link
Author

ghost commented Feb 4, 2013

@jdlx

es geht mir darum für den Fall das man nur Standard Metainfos in nem eigenen tab anzeigen will

was meinst du eigentlich mit Standard Metainfos in nem eigenen Tab? Hier z.B. http://www.redaxo.org/de/forum/wunschliste-f5/editier-modus-metadaten-aktionen-anzeigen-t14727.html hatte ich eine Trennung von MetaInfos und Aktionen (heut würd ichs Funktionen nennen) vorgeschlagen was in R5 schon drin is. Meinst sowas?

Falls man eh an die content.inc.php ranmuss könnte man sich ja überlegen nicht auch hier so ne Trennung durchzuführen. Habe neulich eine Standard-Installtion mit RexSEO gehabt. Unter Metainfos/Sonstiges sah es sehr chaotisch aus: zuerst Artikelname ändern, dann die Meta-Infos der Demo, dann Rexseo Zeugs und dann noch die Kopiergeschichten etc.. Ich könnte mir vorstellen, dass viele Die Defaults der Demo und RexSEO so belassen und dann hier echt ein haufen Zeug drin stehen haben.

@jdlx
Copy link
Contributor

jdlx commented Feb 4, 2013

was meinst du eigentlich mit Standard Metainfos

Einfach Metainfos.. also so wie man sie mit Metainfo anlegt, sprich keine wirklich eigenen pages/hacks.

Unter Metainfos/Sonstiges sah es sehr chaotisch aus

Ja, das tuts schnell.. ist nicht ideal. Liegt imho u.a. daran, daß es da kein autom. prio Management gibt, und man daher mit konkretem Positionieren ein irre Mühe hätte.. d.h. man klemmt sich tendenziell mit ner möglichst großen prio hinter das vorhandene usw.

@staabm: ich hatte die performance nicht gemessen.. mit den normales default metas wars nicht merkbar, aber das kann sich natürlich schnell ändern wenn da sonstwelches Geschichten mit eingeklinkt sind, sprich: ja, ajax wär das Mittel der Wahl. Für r4 wär das Faß aber wieder bischen groß imho.. aber man kann das ja mal für r5 als Idee im Hinterkopf behalten.

@staabm
Copy link
Member

staabm commented Feb 4, 2013

Ich denke das wir an dieser Stelle (Metainfos) im allgemeinen in R4 nichts mehr ändern sollten.

@tbaddade
Copy link
Member

tbaddade commented Feb 4, 2013

@staabm 👍

@jdlx
Copy link
Contributor

jdlx commented Feb 4, 2013

Yo, auch wenn das an sich ne schöne Sache wäre - was man ja für r5 als Idee weiterentwickeln/übernehmen kann - ist die Investition in r4 fragwürdig. Der status quo ist zwar nicht ideal, aber auch weit davon entfernt ein signifikantes prob zu sein. Wenn man das ganz dringend will, kann man das aktuell auch per OPF machen.. wie das tags Addon z.b.

@staabm
Copy link
Member

staabm commented Feb 4, 2013

jop, für post R5.0 können wir hier gerne dort in separatem Issue weiterdiskutieren.

@rexdude gibt es hier dann jetzt noch Punkte des urspr. Issues die diskutiert werden müssen?

@gharlan
Copy link
Member

gharlan commented Feb 4, 2013

Ja, das Ursprungsproblem hier werde ich mir noch anschauen, da wäre es schon schön, wenn wir eine Lösung für hätten.

@ghost
Copy link
Author

ghost commented Feb 4, 2013

Yo, wieder zurück zum Urspungsproblem, sprich. sich in das content menu per include-datei einhacken zu können wäre sau cool, weil halt jetzt schon viele addons davon profitieren würden. sprich eine eigenständige seo page sollte so realisierbar sein und idealerweile auch mit der schnellnavi von be_search zusammenfunzen.

plus evtl. das hier:

Falls man eh an die content.inc.php ranmuss könnte man sich ja überlegen nicht auch hier so ne Trennung durchzuführen. Habe neulich eine Standard-Installtion mit RexSEO gehabt. Unter Metainfos/Sonstiges sah es sehr chaotisch aus: zuerst Artikelname ändern, dann die Meta-Infos der Demo, dann Rexseo Zeugs und dann noch die Kopiergeschichten etc.. Ich könnte mir vorstellen, dass viele Die Defaults der Demo und RexSEO so belassen und dann hier echt ein haufen Zeug drin stehen haben.

@ghost
Copy link
Author

ghost commented Feb 4, 2013

und es wäre natürlich gut zu bestimmen, an welcher stelle im content menu die page rein darf, so wie halt jetzt auch schon, aber momentan kann ja nur der link geändert werden.

@ghost
Copy link
Author

ghost commented Feb 4, 2013

das hier meinte ich vorher:

meta

@dergel
Copy link
Member

dergel commented Feb 6, 2013

hier ein Beispiel für die Verwendung:

rex_register_extension('PAGE_CONTENT_MENU', function ($params) {

    $class = "";
    if ($params['mode']  == 'mysubpage') {
        $class = 'class="rex-active"';
    }

    $params['subject'][] = '<a '.$class.' href="index.php?page=content&amp;article_id=' . $params['article_id'] . '&amp;mode=mysubpage&amp;clang=' . $params['clang'] . '">Meine Subpage</a>';
return $params['subject'];
});


rex_register_extension('PAGE_CONTENT_OUTPUT', function ($params) {
if ($params['mode']  == 'mysubpage') {
return 'bla bla';
}
});

@ghost
Copy link
Author

ghost commented Feb 6, 2013

Vielen Dank!!! Es funzt aber noch nicht ganz :(

Wenn ich das $params Array ausgebe beim zweiten EP steht da imme nur [mode] => edit obwohl oben in der URL mode=mysubpage steht ???

@gharlan
Copy link
Member

gharlan commented Feb 6, 2013

Hast du den aktuellen Stand? @dergel hat noch was commitet kurz vor seinem Kommentar hier…

@ghost
Copy link
Author

ghost commented Feb 6, 2013

ne, hatte nicht den aktuellen Stand...aber jetzt gehts!!! F*ck ist das geil!!! Danke!!!

wenn man jetzt nur noch per be_search schnellnavi hier genauso wie bei den anderens tabs/pages den artikel wechseln kann wäre ich über glücklich :)))

@ghost
Copy link
Author

ghost commented Feb 6, 2013

Und wie ist es eigenlich bei PAGE_CONTENT_MENU mit dem ctype Parameter? Wenn ich das richtig sehe, bekomme ich den nicht über $params um diesen an meine neue URL zu hängen. Passt es hier wenn ich den über rex_get mir hole, oder sollte dieser hier auch noch ergänzt werden? Und as gleiche wäre dann evtl. auch für PAGE_CONTENT_OUTPUT...wenns überhaupt ein Thema is.

Will nur vermeiden, dass mir der aktuelle ctype verloren geht, wenn ich auf ne custom page klicke...

@gharlan
Copy link
Member

gharlan commented Feb 6, 2013

Schnellnavi schaue ich mir an.
ctype: Der Parameter ist ja nur für mode=edit wichtig, oder? Daher kannst du den eigentlich ignorieren, denn wenn man dann wieder in mode=edit möchte wird man ja einen der Ctype-Links anklicken.

@ghost
Copy link
Author

ghost commented Feb 6, 2013

ctype: Der Parameter ist ja nur für mode=edit wichtig, oder?

Ja, das ist korrekt. Aber trotzdem geht dann halt die Ctype-Einstellung des Users verloren spricht er ist auf ctype=3 klickt dann auf die coole neue Subpage und gleichzeitig springt ctype wieder zurück...ist irgendwie nich so schön finde ich....oder?

Schnellnavi schaue ich mir an.

Fein :)))

@dergel
Copy link
Member

dergel commented Feb 6, 2013

auch wenn das nicht vorgesehen ist, kannst du ja in deinem link den ctype übernehmen und dementsprechend weiterverarbeiten.. rex_request( ..

@ghost
Copy link
Author

ghost commented Feb 6, 2013

Ok, passt! :)

@ghost
Copy link
Author

ghost commented Feb 6, 2013

@dergel Du bist heut mein Held des Tages :)

@gharlan
Copy link
Member

gharlan commented Feb 6, 2013

Auch wenn man auf "Metadaten/Sonstiges" klickt, geht der Ctype quasi verloren. Ich finde es aber richtig, dass dann links im Ctype-Menu nichts als aktiv markiert ist, da die Ctypes wie gesagt nur für mode=edit von Belang sind.
Man kommt ja jederzeit mit einem Klick zu seinem Ctype, verstehe nicht, was es für einen Vorteil hat, wenn man sich den letzten Ctype merkt, außer man will auf seiner eigenen Subpage auch etwas Ctype-abhängiges machen…

gharlan added a commit that referenced this issue Feb 6, 2013
@ghost
Copy link
Author

ghost commented Feb 7, 2013

Auch wenn man auf "Metadaten/Sonstiges" klickt, geht der Ctype quasi verloren.

Ja klar, der geht ja verloren. Der er in der URL immer mitübergeben wurd, ging ich davon aus, dass er erhalten bleibt bzw. die Selection. Also passt aufjedenfall :)

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

No branches or pull requests

5 participants