Layout "2 columns" on page "Event Detail" is redundant
Closed this issue · 9 comments
Layoutzuweisung auf Seite Event Detail kann entfernt werden.
Und weil ich gerade den Screenshot hier habe und kein eigenes Ticket dafür aufmachen möchte: Diese Aliase können noch geändert werden:
- My Account (account) => my-account
- Logout (uer-logout) => logout
- RSS Reader (rss) => rss ? Wenn der Alias hier auf rss stehen bleiben soll, könnte man den Seitennamen auch nach RSS ändern
Das Bunte sind die Layouts, Grau die Aliase, Grün die CSS-Klassen.
@Aybee: Danke für die Hinweise. Werd mich dieser Dinge annehmen.
@sascha-mueller wo ist denn der commit zum Fix?
Wenn Du das auf den "close" beziehst, der war nicht beabsichtigt.
War mir garnicht bewußt, dass ich das Ticket geschlossen hatte.
Zur schnellen Kontrolle kannst du die Dateien aus dieser ZIP in deinen ROOT schieben.
http://public.andreasburg.de/MyBE.zip
Das redundante layout hab ich entfernt.
Die beiden Seiten
top-menu-folder/login/account
top-menu-folder/login/user-logout
waren ohne Inhalt und anscheinend auch nirgendwo verlinkt.
Ich habe sie deshalb entfernt.
Bei der RSS-Seite empfinde ich den Titel als "Beschreibendes Element mit "RSS Reader" als korrekt.
Aus meiner Sicht ist der Alias hingegen mit "rss" korrekt benannt. Ich habe mich daher dazu entschlossen diesen nicht zu ändern.
Ich danke dir. RSS könnte man evtl. RSS News(rss-news) benennen.
Ich bin immernoch mit der Benamung zufrieden, wie ich Sie oben beschrieben habe.
Sollten mehrere Leute damit nicht einverstanden sein, wär ich als nächstes dafür es einfach nur rss zu nennen.
Ist ja auch nicht so wichtig, ist halt nur nicht konsistent. Mir als geborenem Kleinkrämer fällt es halt auf ;)
Die Seiten mit dem Newsreader und dem Eventreader heißen ja auch nicht "News Reader"... Und es gibt auch sonst keine Seiten, welche nach ihrem Haupt-Modul benannt sind. Und wenn sie doch "RSS Reader" heißt, was spricht dann hier dafür auf die Konsistenz zu verzichten und diese Seite als einzige nicht "rss-reader.html5" zu benennen. Technische Hintergründe?
Nur RSS scheint mir nicht eindeutig. Beim RSS Button rechts oben hatte ich z.B. erwartet das der RSS-Feed mit den News der Demo-Seite geoffnet wird.
Was kam würde ich als "Contao Project News (RSS-Reader)" bezeichnen.