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

Design changes #2414

Merged
merged 1 commit into from
Nov 14, 2024
Merged

Conversation

davidtrussler
Copy link
Contributor

@davidtrussler davidtrussler commented Nov 12, 2024

Trello

The changes in this PR make a couple of small changes on the Edit and Metadata tabs (screenshots below). The reason for the changes to the label/legend elements on the Metadata tag is to be consistent across Publisher and render these elements just as a <label> or <legend> without wrapping them in a header tag which is better for accessibility.

  • Edit:
    • Increases the depth of the body field
    • Adds a rule above change note section (does not display when the change note is not rendered)
  • Metatags:
    • remove headings from label/legend (no visual changes - removes the <h3> tags from the label and legend)
Current Updated
Admin tab Admin tab
Screenshot 2024-11-12 at 13 16 59 Screenshot 2024-11-12 at 13 16 44
Metadata tab Metadata tab
Screenshot 2024-11-12 at 13 06 19 No visual change

@davidtrussler davidtrussler force-pushed the 407_Design-System_Edit-tab_design-updates branch from c7f1425 to 20370cf Compare November 12, 2024 13:20
- Edit:
  - Increase depth of body field
  - Add rule above change note section
- Metatags:
  - remove headings from label/legend
@davidtrussler davidtrussler force-pushed the 407_Design-System_Edit-tab_design-updates branch from 20370cf to 2f67f6e Compare November 12, 2024 13:21
@davidtrussler davidtrussler marked this pull request as ready for review November 14, 2024 12:00
Copy link
Contributor

@mtaylorgds mtaylorgds left a comment

Choose a reason for hiding this comment

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

LGTM

@davidtrussler davidtrussler merged commit acf27ec into main Nov 14, 2024
12 checks passed
@davidtrussler davidtrussler deleted the 407_Design-System_Edit-tab_design-updates branch November 14, 2024 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants