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

Improve notices style #34901

Closed
javierarce opened this issue Sep 17, 2021 · 10 comments
Closed

Improve notices style #34901

javierarce opened this issue Sep 17, 2021 · 10 comments
Labels
[Feature] UI Components Impacts or related to the UI component system Needs Accessibility Feedback Need input from accessibility Needs Design Feedback Needs general design feedback. [Type] Enhancement A suggestion for improvement.

Comments

@javierarce
Copy link
Contributor

javierarce commented Sep 17, 2021

Description

While working on this issue I noticed that we have a problem with the, well, notices. The current implementation differs noticeably (pun intended) from the original design.

Here's a comparison between the implementation (left) and the ideal version (right)

image

Here are the things I think we should fix:

  • The line height and text color
  • The color and position of the X
  • Internal paddings (review)

The color differences are especially important because they have lower contrast than the designed version.

Step-by-step reproduction instructions

You can show an error notice enabling the Navigation experiment and creating a menu with an empty or a duplicate name:

  1. Go to Gutenberg > Experiments.
  2. Enable the Navigation screen experiment.
  3. Create a menu with the name "Hello"
  4. Repeat step 3 to create a duplicated menu.

Screenshots, screen recording, code snippet

No response

Environment info

No response

Please confirm that you have searched existing issues in the repo.

Yes

Please confirm that you have tested with all plugins deactivated except Gutenberg.

Yes

@javierarce javierarce added Needs Design Feedback Needs general design feedback. [Feature] UI Components Impacts or related to the UI component system Needs Accessibility Feedback Need input from accessibility labels Sep 17, 2021
@javierarce
Copy link
Contributor Author

javierarce commented Sep 17, 2021

The notice that appears when there is a backup post looks better, by the way, ad I see why the X is not really black: that color is reserved for the hover version.

Screenshot 2021-09-17 at 11 57 42

I now wonder if the problem with the notice error is only related to that specific location I mentioned in the original issue.

@javierarce
Copy link
Contributor Author

Here's another problematic notice that appears when you try to upload an unsupported file to the Media Block (the actual text should also be improved, by the way)

Screenshot 2021-09-17 at 12 13 59

@Mamaduka
Copy link
Member

The "X" button position is my fault in Navigation Screen. I will fix it in follow-up PR. 🙇

@javierarce
Copy link
Contributor Author

javierarce commented Sep 17, 2021

Thanks, @Mamaduka!… I wasn't sure if that was a localized problem or something more general, I should have looked for more examples before creating this issue :/

@Mamaduka
Copy link
Member

@javierarce fixed the button position issue in this PR #34903.

@javierarce
Copy link
Contributor Author

@Mamaduka thanks! I reviewed it and saw that the fix works.

I'll leave this issue open to address the other problems I described.

@karmatosed
Copy link
Member

@javierarce just to clarify I think there might be some confusion of the wp-admin notices and editor ones - there are different ones. For example, the border to the side is one used in certain cases there. Overall in WP notices need some unification, so I am trying though to work out what after the fixes is left to work on here, could you perhaps pull out a list of what remains to help?

@javierarce
Copy link
Contributor Author

Overall in WP notices need some unification, so I am trying though to work out what after the fixes is left to work on here, could you perhaps pull out a list of what remains to help?

I hadn't planned anything beyond fixing the issues I described in the first message.

@talldan
Copy link
Contributor

talldan commented Sep 20, 2021

related - #33853

There's actually no implementation of a blue info notice at the moment, so that might be something to address alongside the styling fixes.

@Mamaduka
Copy link
Member

Closing in favor of #67662.

@Mamaduka Mamaduka closed this as not planned Won't fix, can't repro, duplicate, stale Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] UI Components Impacts or related to the UI component system Needs Accessibility Feedback Need input from accessibility Needs Design Feedback Needs general design feedback. [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

No branches or pull requests

5 participants