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

UI: Fix admin headings #965

Merged
merged 1 commit into from
Jul 29, 2023
Merged

UI: Fix admin headings #965

merged 1 commit into from
Jul 29, 2023

Conversation

GaryJones
Copy link
Contributor

Description

  • Use h1 instead of h2 for admin page titles.
  • Remove the redundant icon divs.
  • Implement page title action in the correct way so that it visually appears next to the page title, but isn't a part of it.

Deploy Notes

None.

Steps to Test

Apply patch and see that the bullets above are now correct.

- Use h1 instead of h2 for admin page titles.
- Remove the redundant icon divs.
- Implement page title action in the correct way so that it visually appears next to the page title, but isn't a part of it.
@GaryJones GaryJones added this to the 3.5.12 milestone Jul 29, 2023
@GaryJones GaryJones self-assigned this Jul 29, 2023
@GaryJones GaryJones merged commit 6229d6d into master Jul 29, 2023
10 checks passed
@GaryJones GaryJones deleted the fix/admin-headings branch July 29, 2023 22:41
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.

1 participant