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

Fabrique branding [WIP] #77

Draft
wants to merge 1 commit into
base: release/6.x
Choose a base branch
from
Draft

Conversation

7ochem
Copy link
Contributor

@7ochem 7ochem commented Mar 23, 2023

⚠️ Work In Progress ⚠️


Changed

  • 1️⃣ Admin login page restyling ⚡
  • 2️⃣ Sonata Admin top logo and title 👍🏻
  • 3️⃣ Front side admin menu 💻

Admin login page restyling

Before:

Screenshot from 2023-03-23 11-54-09

Proposal 1:

Zicht-CMS-login_Fab-branded
Based on styles from https://www.fabrique.nl/ and specifically https://www.fabrique.nl/bel-mij-terug/
TODO:

  • Check with designers 😁
  • Fix login page <title>

Sonata Admin top logo and title

Before

Screenshot from 2023-03-23 11-47-25

Now

Screenshot from 2023-03-23 11-46-40


Front side admin menu

Before

Screenshot from 2023-03-23 11-48-56

Now

Screenshot from 2023-03-23 11-49-19

Same style as in the Craft CMS projects.
TODO

  • Fix the 8px gap between logo and folded out menu

Other TODO's

  • Update CHANGELOG.md
  • After PR merge: merge to release 7 and 8
  • After release: update projects (what will be the process, should we inform clients (who will get a totally different login screen)

- Admin login page restyling
- Sonata Admin top logo and title
- Front side admin menu
@@ -6,44 +6,47 @@
<title>Zicht CMS | login</title>

Choose a reason for hiding this comment

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

Kunnen meteen ook dit even wijzigen ;)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oeh 🙈 Goeie!

Copy link
Contributor

@VMeijer-Zicht VMeijer-Zicht left a comment

Choose a reason for hiding this comment

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

Hartstikke leuk! 😄

Copy link
Contributor

@avanderhoek avanderhoek left a comment

Choose a reason for hiding this comment

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

Lekker bezig Jochem!

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

Successfully merging this pull request may close these issues.

4 participants