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

Inconsistent page alert placement #3286

Open
kristinacyin opened this issue Jan 1, 2025 · 0 comments
Open

Inconsistent page alert placement #3286

kristinacyin opened this issue Jan 1, 2025 · 0 comments
Labels
bug Something that isn't working as intended dev issue is for the dev team Feature: 🏢 Org Model

Comments

@kristinacyin
Copy link
Contributor

kristinacyin commented Jan 1, 2025

Current Behavior

In org model, some page alerts are above the breadcrumb in manage pages while others are under. Namely, success (and info) alerts are above the breadcrumb, while error alerts are below.

Below example
Image

Above example:
Image

Expected Behavior

All the page alerts should be above the breadcrumb. Like so:

Image

Please check all the page alerts to make sure they are placed above the breadcrumb.

Steps to Reproduce

  1. Turn on org feature waffle flag
  2. Input wrong answers to fields in the manage domain pages

Environment

No response

Additional Context

According to the Figma design, there should be a 16px margin between the page alert and the breadcrumb. The page alert should be directly adjacent to the side navigation bar, and have same top edge line.

Image

Issue Links

No response

@kristinacyin kristinacyin added bug Something that isn't working as intended dev issue is for the dev team labels Jan 1, 2025
@abroddrick abroddrick moved this from 👶 New to 🎯 Ready in .gov Product Board Jan 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something that isn't working as intended dev issue is for the dev team Feature: 🏢 Org Model
Projects
Status: 🎯 Ready
Development

No branches or pull requests

2 participants