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

Layout - Order of frontend modules reversed #861

Closed
contaoacademy opened this issue Oct 21, 2019 · 16 comments
Closed

Layout - Order of frontend modules reversed #861

contaoacademy opened this issue Oct 21, 2019 · 16 comments
Labels
Milestone

Comments

@contaoacademy
Copy link

Affected version(s)
4.4.44

Description
In the layout, the columns for frontend modules are displayed in reverse order.

Previously: Header > Main > Footer
Now: Footer > Main > Header

See screenshot
module

@mlwebworker
Copy link

Ich finde ja die alphabetische Sortierung hat auch etwas für sich. Früher habe ich mich immer geärgert, dass Custom-Bereiche ganz unten landen. Das dürfte damit ja auch der Vergangenheit angehören.

@contaoacademy
Copy link
Author

Ich finde ja die alphabetische Sortierung hat auch etwas für sich.

Ich finde es so unlogisch sortiert. Unter Contao 4.8 ist es noch so wie früher und ich sehe darin einen Bug.

@xchs
Copy link
Contributor

xchs commented Oct 21, 2019

@leofeyer
Copy link
Member

Ich sehe darin keinen Bug.

Vor Contao 4.8 waren die eigenen Layoutbereiche immer am Ende und jeder, dessen Layoutbereich mit "A" begann, hat sich gewundert, warum dieser nicht angezeigt wird (weil eben erwartet wurde, dass "A" oben steht). Jetzt sind die Bereiche alphabetisch geordnet und der Bereich mit "A" wird wie erwartet oben angezeigt.

Natürlich stolpert man als erfahrener Contao-Nutzer vielleicht darüber, weil es halt anders ist als vorher. Aber anders heißt ja nicht automatisch schlechter.

@contaoacademy
Copy link
Author

contaoacademy commented Oct 25, 2019

@leofeyer
Hmm, es ist vielleicht kein Bug.
Aber das Verhalten zwischen Contao 4.44.4 und Contao 4.8.4 ist trotzdem anders.
Unter 4.4 wie im 1. Post (Fußzeile vor Kopfzeile) unter 4.8.4 so wie ich das kenne (Kopfzeile vor Fußzeile) (siehe Screenshot)
contao-4-8

@contaoacademy
Copy link
Author

Ergänzung…
Im Grunde ist es mir egal. Hauptsache bei Contao 4.8.4 verhält es sich dann gleich wie unter 4.4.44

@Mynyx
Copy link
Contributor

Mynyx commented Oct 25, 2019

Vor Contao 4.8 waren die eigenen Layoutbereiche immer am Ende und jeder, dessen Layoutbereich mit "A" begann, hat sich gewundert, warum dieser nicht angezeigt wird (weil eben erwartet wurde, dass "A" oben steht).

Vielleicht die eigenen Layoutbereiche am Anfang anzeigen und darunter die Standardlayoutbereiche in logischer Reihenfolge.

@aschempp
Copy link
Member

Ich sehe nur zwei Möglichkeiten:

  1. Alphabetisch (einfach umzusetzen / bereits der Fall)
  2. Wie konfiguriert. Das wäre imho am sinnvollsten. Die Layoutbereiche haben eine definierte Position, entsprechend könnten sie auch in der Liste sein.

@leofeyer leofeyer added the bug label Oct 28, 2019
@leofeyer leofeyer added this to the 4.4 milestone Oct 28, 2019
@contaoacademy
Copy link
Author

2. Wie konfiguriert. Das wäre imho am sinnvollsten. Die Layoutbereiche haben eine definierte Position, entsprechend könnten sie auch in der Liste sein.

Ich denke, das wäre doch die logische Variante. Damit habe ich die volle Kontrolle über die Position der Module und kann mir das so aufbauen wie es mir gefällt.

@leofeyer
Copy link
Member

Hm, also in Contao 4.8 scheinen die Bereiche bereits nach definierter Position sortiert zu sein; wir haben das offenbar als neues Feature in der 4.6 hinzugefügt (siehe 94de42a bzw. contao/core-bundle#1529).

@contao/developers Wenn das ein neues Feature in der 4.6 war, bedeutet das "wontfix" für dieses Issue in Contao 4.4 oder soll ich die Änderungen zurückportieren?

@contaoacademy
Copy link
Author

Ich wäre für "Änderungen zurückportieren".

@Toflar
Copy link
Member

Toflar commented Oct 30, 2019

wontfix.
Für mich ist das klar für den Betrieb irrelevant und kein Bug.
Die 4.9 steht eh vor der Tür - ich möchte mich ungern mit solchen Details aufhalten.

@ausi
Copy link
Member

ausi commented Oct 30, 2019

wontfix

@contaoacademy
Copy link
Author

contaoacademy commented Oct 31, 2019

Die 4.9 steht eh vor der Tür - ich möchte mich ungern mit solchen Details aufhalten.

Da stimme ich dir zu.
Ich schließe das Ticket, da das Verhalten unter Contao 4.9 ja dann wie bei Contao 4.8 sein wird.

@aschempp
Copy link
Member

Für mich ist das klar für den Betrieb irrelevant und kein Bug.

Ich habe leider gestern festgestellt, dass das nicht so ist. Durch die geänderte Reihenfolge hat sich auch die Reihenfolge der Generierung im Frontend verändert. Früher wurden Header-Module immer vor den Main-Modulen generiert, darauf verlässt sich z.B. Isotope. Durch die Sortierung nach Name ist das nicht mehr so bzw. sogar "zufällig" je nach Sprache.

Noch schlimmer daran ist, dass die Sortierung beim anlegen beibehalten wird! Erst nach einem erneuten Speichern der Darstellung ist die Reihenfolge in der DB alphabetisch…

@leofeyer
Copy link
Member

leofeyer commented Dec 3, 2019

See #1042

@leofeyer leofeyer closed this as completed Dec 3, 2019
leofeyer added a commit that referenced this issue Dec 3, 2019
Description
-----------

Fixes #861

Commits
-------

e2178ef Sort the custom layout sections by their position (see #861)
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 18, 2024
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

8 participants