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

[5.x]: UI and layout issues with cp header and sidebar #14900

Closed
mlathrom opened this issue Apr 30, 2024 · 2 comments · Fixed by #14910
Closed

[5.x]: UI and layout issues with cp header and sidebar #14900

mlathrom opened this issue Apr 30, 2024 · 2 comments · Fixed by #14910
Assignees
Labels

Comments

@mlathrom
Copy link
Contributor

mlathrom commented Apr 30, 2024

What happened?

Description

I think the layout of the header and sidebar on control panel pages could be cleaner. There are also some spacing inconsistencies in at different breakpoints.

Expected behavior

  • Clean layout at all resolutions
  • Skip to Entries button should remain in sidebar element
  • Toolbar should always line up with main content

Actual behavior

  • "Skip to Entries" button is out of the sidebar on mobile
  • Toolbar, action buttons, and show sidebar interface is a bit clunky in it's layout
  • The toolbar doesn't line up with the main content in tablet resolution

Mobile Layout

Lots of vertical space used.

mobile-layout

Mobile Layout

Skip to entries button out of sidebar container.

mobile

Tablet layout

Toolbar is not aligned with main content. Strange spacing.

tablet

Desktop Layout

Odd spacing for Skip to Entries button

desktop

Craft CMS version

5.05

PHP version

8.2

Operating system and version

No response

Database type and version

No response

Image driver and version

No response

Installed plugins and versions

@mlathrom
Copy link
Contributor Author

@brianjhanson I've created a fix for this. Hope it helps.

#14901

@brandonkelly
Copy link
Member

Craft 5.10.0-beta.1 is out with changes from #14901 and #14910. Thanks for the feedback and PR!

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

Successfully merging a pull request may close this issue.

3 participants