Beta 2.0 - Statische Seiten

Hier können Probleme und alles andere in Deutscher Sprache gelöst werden.
Timbalu
Regular
Posts: 4598
Joined: Sun May 02, 2004 3:04 pm

Re: Beta 2.0 - Statische Seiten

Post by Timbalu »

(Siehe related category association problem in last post)
Timbalu wrote:Ich persönlich würde wahrscheinlich eher die multiple Assoziierung verbieten, soweit das mit vertretbarem Aufwand möglich ist.
Nein ist es nicht. Also habe ich mich für jetzt dagegen entschieden und lieber eine erweiterte Info eingebaut.
Regards,
Ian

Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: Beta 2.0 - Statische Seiten

Post by garvinhicking »

Timbalu wrote:(Siehe related category association problem in last post)
Timbalu wrote:Ich persönlich würde wahrscheinlich eher die multiple Assoziierung verbieten, soweit das mit vertretbarem Aufwand möglich ist.
Nein ist es nicht. Also habe ich mich für jetzt dagegen entschieden und lieber eine erweiterte Info eingebaut.
Hihi, ok. Sowas wollte ich gerade schreiben, dass der Aufwand da vermutlich schon recht hoch ist :)
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
Timbalu
Regular
Posts: 4598
Joined: Sun May 02, 2004 3:04 pm

Re: Beta 2.0 - Statische Seiten

Post by Timbalu »

Dies unklare Geschichte stört mich trotzdem immer noch.

Fakt ist, ohne einen Eintrag in die *_staticpage_categorypage Tabelle via den related category hook im Categoryform und dem expliziten Patch der entries tpl, haben diese related category verknüpften Seiten keinen Sinn.

Das Select für die zugeordnete Kategorie im Staticpageform hat so bisher nur informativen Charakter und das auch nur, wenn man der staticpage Seite die Information über die zugewiesenen Kategorie über dasselbe schon einmal mitgegeben hat.

Für mich als "Anwender" stellte sich das Auswahlfeld bisher aber immer so dar, dass ich selber einer Seite einer Kategorie tatsächlich zuweisen kann.

Um das auch stattfinden zu lassen, muss einer Verknüpfung in diesem Auswahlfeld auch eine Eintragung in die *_staticpage_categorypage Tabelle folgen, so wie es das Categoryform macht. Dort gibt es aber nur 1:1 Beziehungen. Also überschreibt natürlich jede andere statische Seite, mit derselben Kategorie verknüpft, die vorherige.

Das könnte man kommunizieren und die ganze Sache wäre endlich klar.
(Oder übersehe ich etwa irgendwelche anderen Beziehungen?)

Thoughts?
Regards,
Ian

Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: Beta 2.0 - Statische Seiten

Post by garvinhicking »

Hi!

Ich bin ab Mittwoch in Urlaub und habe gerade auf der Arbeit noch einige Projekte zu übergeben. Ich komme vermutlich früher nich tmehr dazu mir das im Code noch einmal wirklich anzuschauen. Grundsätzlich kann es durchaus sein dass es hier zwei kollidierende Faktoren gibt, aber andererseits wundere ich mich ob es nicht doch auch einen genauen Grund für die duplikation von categorypage-tabelle vs. Spalte gibt.

Wenn das aber der de-facto zustand derzeit ist, würde ich sagen - tatsächlich kommunizieren dass es eine 1:1 einstellung ist, und wenn es im code doch anders ist müssen wir halt nochmal ran....
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
Timbalu
Regular
Posts: 4598
Joined: Sun May 02, 2004 3:04 pm

Re: Beta 2.0 - Statische Seiten

Post by Timbalu »

Das wäre relativ einfach in updateStaticPage()

Code: Select all

// update staticpage_categorypage table with set data, if related_category_id was set in form on update
if (!empty($insert_page['related_category_id']) && $insert_page['related_category_id'] > 0) {
    $data = array(
            'categoryid '               => (int)$insert_page['related_category_id'],
            'staticpage_categorypage'   => (int)$pid,
    );
    // RQ: Note this to user, that this will change an already other set category related staticpage ?
    //$categorypage = $this->fetchCatProp((int)$insert_page['related_category_id']);
    //if ($categorypage > 0 && $categorypage != $insert_page['id']) say hello or proceed and throw message note

    $this->setCatProps((int)$insert_page['related_category_id'], $data);
    unset($data);
}
und dann halt beim Auswahlfeld per title oder info button eine kurze Erklärung zur 1:1 Beziehung.
Regards,
Ian

Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
Timbalu
Regular
Posts: 4598
Joined: Sun May 02, 2004 3:04 pm

Re: Beta 2.0 - Statische Seiten

Post by Timbalu »

garvinhicking wrote:Grundsätzlich kann es durchaus sein dass es hier zwei kollidierende Faktoren gibt, aber andererseits wundere ich mich ob es nicht doch auch einen genauen Grund für die duplikation von categorypage-tabelle vs. Spalte gibt.

Wenn das aber der de-facto zustand derzeit ist, würde ich sagen - tatsächlich kommunizieren dass es eine 1:1 einstellung ist, und wenn es im code doch anders ist müssen wir halt nochmal ran....
Ich wäre dir sehr verbunden, wenn du dir das daraufhin noch mal anschaust.
Inzwischen habe ich - leider mit erheblichem Aufwand - diese Assoziierung hergestellt und commited, würde das als tagged release aber erst freigeben, wenn du keinen anderen Grund findest.
Regards,
Ian

Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: Beta 2.0 - Statische Seiten

Post by garvinhicking »

Hey,

klar. Wo kann ich mir den diff anschauen?

LG,
Garvin
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
Timbalu
Regular
Posts: 4598
Joined: Sun May 02, 2004 3:04 pm

Re: Beta 2.0 - Statische Seiten

Post by Timbalu »

Hi. Gut erholt? :)

Dabei ging es nicht um ein Diff, sondern darum, dass du dir den "alten" code daraufhin anschauen wolltest, um vielleicht doch noch einen anderen Grund freizulegen.

(Wie das dann mit dem "Fix" funktioniert, können wir uns demnächst ja dann auf unserem Dev Blog anschauen.)
Regards,
Ian

Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: Beta 2.0 - Statische Seiten

Post by garvinhicking »

Achso. Ich hatte mir das im alten Code ja schon angesehen und da jetzt auch nicht wirklich eine Möglichkeit gesehen, 1:n zu nutzen. Insofern bin ich schon für die Änderung so dass 1:1 für den Benutzer klar kommuniziert ist.
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
Timbalu
Regular
Posts: 4598
Joined: Sun May 02, 2004 3:04 pm

Re: Beta 2.0 - Statische Seiten

Post by Timbalu »

Das Staticpages Dev Plugin hat für mich seit dem 1. Dezember einen RC Status erreicht.

Ich wäre also froh um jeden, der dies mit (s)einer Serendipity 1.7 oder kommenden 2.0 RC Installation testen würde. Download hier: https://github.com/ophian/serendipity_e ... /tag/v4.30

Ich meine, dass keine besondere Sicherheitskopie der DB oder des vorherigen Plugins nötig wäre, muss allerdings darauf hinweisen, dass diese Version bisherige Verhaltenweisen von bestehenden und neuen Seiten verändern kann, sei es aus bugfix Gründen, oder aus korrigiertem unstimmigen Verhalten.
Dazu zählen u.a.
  • related category relations, which now allow 1:1 relations only
  • staticpage and breadcrumb navigations (fixes, designs and effects templates plugin_staticpage.tpl)
  • more strict database query selects
  • new backend template forms (may effect your template files too)
  • etc
Wie ich die nötigen Änderungen der plugin_staticpage.tpl für bulletproof und 2k11 allgemein verfügbar mache, weiß ich noch nicht. Ich habe sie für diese beiden themes aber als zip ebenso bereitgestellt.
Regards,
Ian

Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
Timbalu
Regular
Posts: 4598
Joined: Sun May 02, 2004 3:04 pm

Re: Beta 2.0 - Statische Seiten

Post by Timbalu »

Staticpages Dev Plugin 4.33 https://github.com/ophian/serendipity_e ... /tag/v4.33
mit angepassten templates fur 2k11/bulletproof.
Regards,
Ian

Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: Beta 2.0 - Statische Seiten

Post by garvinhicking »

Hi Ian!

Ich gebe das Feedback einfach mal hier, damit wir auf deutsch problemlos besprechen können und kein extra issue benötigen:

- Ich bin noch nicht so ganz glücklich mit dem |escape der Titel in den TPL-Files mit Benutzung von staticpage_doublesc. Warum genau ist das notwendig? Sorry dass ich darauf nochmal rumreiten muss, aber ich dachte wir hätten zuletzt in den Threads dazu entschieden, wer UTF-8 entities in einem ISO-Blog unbedingt nutzen möchte, sollte sein Blog auf UTF-8 fahren? Gut möglich dass ich hier wieder etwas vergessen habe, daher muss ich nochmal nachhaken. Ich würde zugunsten der Lesbarkeit von Templates wirklich auf den Rattenschwanz an |escape-Optionen gerne verzichten.

- Die ein/ausklappbare Staticpage-Nav ist ganz nett, würde ich aber nicht mit "Show/Hide TabBar" bezeichnen. Ich würde einfach so einen rauf/runterpfeil direkt links in der oberen Navigationszeile (vor "Seiten bearbeiten") einfügen, dann ist das toggeln IMO sofort klar. Als Title-Attribut könnte man dem gerne ein "Toggle backend navigation toolbar" mitgeben?

- Wenn man z.b. im Backend "Zeige META FELD Optionen" (btw, das müsste "Zeige META-FELD-Optionen" korrekt heißen, und "Zeige STRUKTUR-Optionen"!) einblendet, wird diese Zeile so "eingedrückt". Im restlichen 2k11 Backend bleiben die Einklapp/ausklapp-Felder immer im nicht eingedrückten "Button"-Look, das sollte man IMO konsistent halten. Auch sieht das Padding und der generelle Look dieser Buttons anders aus als in 2k11, das wäre schön wenn man das harmonisiert.

- Die Struktur-Optionen sind wohl derart in ein fieldset o.ä. gewrappt, dass es im gegensatz zum darüberliegenden Feld eine unterschiedliche Breite der WYSIWYG-Editoren gibt, dadurch sieht die Toolbar unterschiedlich formatiert aus. Da wäre es schön, wenn das dieselbe Breite hat wie im Container darüber?

- Das Highlighting der aktiven Seite in der Navigation sieht etwas strange aus: https://www.dropbox.com/s/5ubnrvcqko3t0 ... 3.png?dl=0 entweder müssten die Randabstände identisch sein, oder ich würde ggf. einfach den aktiven Button fetten oder so?

VG,
Garvin
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
Timbalu
Regular
Posts: 4598
Joined: Sun May 02, 2004 3:04 pm

Re: Beta 2.0 - Statische Seiten

Post by Timbalu »

garvinhicking wrote: Benutzung von staticpage_doublesc
Nun ja, es fehlt ja eigentlich nur der zusätzliche Optionsknopf um das aktiv zu setzen (und das sowieso nur auf ISO-8859-1 Umgebungen. (Das mit der mangelnden Lesbarkeit für die paar wenigen Stellen finde ich nun etwas an den Haaren herbeigezogen!)
garvinhicking wrote:- Die ein/ausklappbare Staticpage-Nav ist ganz nett, würde ich aber nicht mit "Show/Hide TabBar" bezeichnen. Ich würde einfach so einen rauf/runterpfeil direkt links in der oberen Navigationszeile (vor "Seiten bearbeiten") einfügen, dann ist das toggeln IMO sofort klar. Als Title-Attribut könnte man dem gerne ein "Toggle backend navigation toolbar" mitgeben?
Ja, das wäre nett (gewesen) wenn der klitzekleine Umstand nicht zu vergessen wäre, dass man keinen toggle button dorthin setzt, wo etwas verschwinden (aber auch wiederkommen) soll... :wink:
Deswegen sitzt der unten ( und ist mit Text versehen, aber darüber läßt sich reden ).
garvinhicking wrote: eingedrückten "Button"-Look ... [und] ... unterschiedliche Breite der WYSIWYG-Toolbar
Tatsächlich! Manche Sachen brauchen halt ein frisches Auge! Ersteres war aber zur Entschuldigung auf das Backend css zurückzuführen.
garvinhicking wrote:müssten die Randabstände identisch sein
Das sind sie komischerweise auch. Jedenfalls habe ich noch nicht gemerkt, wodurch dieser Eindruck entstehen kann.
Regards,
Ian

Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
garvinhicking
Core Developer
Posts: 30022
Joined: Tue Sep 16, 2003 9:45 pm
Location: Cologne, Germany
Contact:

Re: Beta 2.0 - Statische Seiten

Post by garvinhicking »

Hi!
Timbalu wrote:
garvinhicking wrote: Benutzung von staticpage_doublesc
Nun ja, es fehlt ja eigentlich nur der zusätzliche Optionsknopf um das aktiv zu setzen (und das sowieso nur auf ISO-8859-1 Umgebungen. (Das mit der mangelnden Lesbarkeit für die paar wenigen Stellen finde ich nun etwas an den Haaren herbeigezogen!)
Also ich finde es schon extrem unleserlich so, und immerhin ist das ja eine Datei die von Template-Autoren auch em ehesten mal angepackt werden könnte. ISO-8859-1 ist ja nun wirklich eine Krücke für Bestandsblogs, wir sollten da jetzt nicht anfangen NEUE Sachen die es bisher nicht gab dafür einzubinden. Wer UTF-8 Zeichen da haben will soll halt ein UTF-8 Blog aufsetzen, dafür müssen wir es nicht auf dem Rücken aller anderen austragen die TPL-Dateien konfuser zu gestalten und Parameter zu nutzen die dann individuell gesetzt werden und nicht zentral per wrapper mal änderbar wären.

Ja, das wäre nett (gewesen) wenn der klitzekleine Umstand nicht zu vergessen wäre, dass man keinen toggle button dorthin setzt, wo etwas verschwinden (aber auch wiederkommen) soll... :wink:
Deswegen sitzt der unten ( und ist mit Text versehen, aber darüber läßt sich reden ).
Der Togglepfeil steht dann natürlich an der Stelle wo die Navi wegklappt, Scherzkeks. :) Ich finde es halt im Footer unschön, etwas zu toggeln was an ganz anderer Stelle erscheint ist UI-mäßig nicht so toll.
garvinhicking wrote:müssten die Randabstände identisch sein
Das sind sie komischerweise auch. Jedenfalls habe ich noch nicht gemerkt, wodurch dieser Eindruck entstehen kann.
Durch die Border innerhalb des Fieldsets, achte da mal drauf?

LG,
Garvin
# Garvin Hicking (s9y Developer)
# Did I help you? Consider making me happy: http://wishes.garv.in/
# or use my PayPal account "paypal {at} supergarv (dot) de"
# My "other" hobby: http://flickr.garv.in/
Timbalu
Regular
Posts: 4598
Joined: Sun May 02, 2004 3:04 pm

Re: Beta 2.0 - Statische Seiten

Post by Timbalu »

garvinhicking wrote:
Ja, das wäre nett (gewesen) wenn der klitzekleine Umstand nicht zu vergessen wäre, dass man keinen toggle button dorthin setzt, wo etwas verschwinden (aber auch wiederkommen) soll... :wink:
Deswegen sitzt der unten ( und ist mit Text versehen, aber darüber läßt sich reden ).
Der Togglepfeil steht dann natürlich an der Stelle wo die Navi wegklappt, Scherzkeks. :) Ich finde es halt im Footer unschön, etwas zu toggeln was an ganz anderer Stelle erscheint ist UI-mäßig nicht so toll.
Ich verstehe dich schon richtig, mein aber bezieht sich aber darauf, dass ich nicht einen Platz ersetzen, sondern komplett verschwinden lassen will. Sonst hat die Funktion keinen Sinn.
Man könnte diesen Toggle Knopf sonst nur and den Title oder die Nav, beides right-flowed ansetzen.
Wenn man nicht entrylisting als default wählt, sitzt er im Block des Selects.
garvinhicking wrote:Durch die Border innerhalb des Fieldsets, achte da mal drauf?
Trotzdem sind die Abstände fix bzw überall gleich. Das Einzige was im aktiven Zustand dazukommt ist der Border.
Regards,
Ian

Serendipity Styx Edition and additional_plugins @ https://ophian.github.io/ @ https://github.com/ophian
Post Reply