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

New Layout #812

Closed
wants to merge 36 commits into from
Closed

New Layout #812

wants to merge 36 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Oct 7, 2023

Aus Langerweile einfach mal bissel rumgespielt, ist zwar noch nicht alles fertig....aber läuft trotzdem schon.

PrivatePHPCoder and others added 21 commits October 5, 2023 17:41
Config.php, damit das Layout installiert werden kann.
HTML Grundgerüst für Ilch-CMS und Testausgabe
Header-Element hinzugefügt und am oberen Bildschirmrand fixiert
(fixed-top)
Kleinen Fehler beseitigt.
CSS-Datei angelegt mit generellen Einstellungen für das Layout ansich.
Flexbox-Container angelegt. Items im Container werden zentriert.
Ilch-Logo hinzugefügt und verlinkt
Der Header bekommt einen inneren Abstand (padding) oben und unten, sowie eine Hintergrundfarbe.
Die Farbe etwas angepasst an Ilch ;)
Später hat man die Wahl zwischen LogoGrafik oder einfachem Text, deswegen das ToDo:
Farbe für Logo-Link
Maximale Größe für Logo-Image festgelegt.
Es wurden 2 Menüs hinzugefügt.

Menü 1, zeigt Menüpunkte separat an.
Menü 2, zeigt das Menü als Dropdown an.
Don't try to create a thumbnail if getimagesize returns false. Either damaged image or no image at all.
@blackcoder87
Copy link
Member

Bitte nicht in den Branch mergen. Der Branch soll später in den master überführt werden und das soll nicht heißen, dass das Layout dann auch direkt fester Bestandteil von Ilch wird. Das hat also eher nichts in diesem Branch zu suchen. Vielleicht sogar das falsche Repository.

@blackcoder87 blackcoder87 marked this pull request as draft October 7, 2023 17:14
Remove call of getDialogMessage in the dialog action as it seems to be not used in the view.
composer.json Outdated Show resolved Hide resolved
blackcoder87 and others added 4 commits October 8, 2023 17:32
Codeverbesserungen.
Version von Bootstrap angepasst, Zahlendreher :(
Menüs haben nun einen schönes Styling
Update Newsletter boxes
Update Job navi.php and show.php
Update Birthday
Update Vote
Update Event
application/layouts/rtx_layout01/config/config.php Outdated Show resolved Hide resolved
application/layouts/rtx_layout01/index.php Outdated Show resolved Hide resolved
application/layouts/rtx_layout01/index.php Outdated Show resolved Hide resolved
application/layouts/rtx_layout01/js/main.js Outdated Show resolved Hide resolved
@ghost ghost marked this pull request as ready for review October 13, 2023 17:21
Copy link
Member

@blackcoder87 blackcoder87 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sofern das Layout kein fester Bestandteil von Ilch wird, gehört es nicht ins Ilch-Repo. Bis das entschieden wurde, blocke ich den PR hiermit.

@ghost ghost locked and limited conversation to collaborators Oct 17, 2023
@ghost ghost closed this Oct 17, 2023
@ghost
Copy link
Author

ghost commented Oct 17, 2023

Alles klar...schließe den PR dann.

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

Successfully merging this pull request may close these issues.

3 participants