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
Small design tweaks to apply #8035
Comments
@mnapoli when there is enough space to fit both,
Example on wide screen: |
My bad, this is not a bug actually (I had created a new Segment locally in a file causing this extra empty category to display)
👍 |
I've given another try to this: This is just too messy to make it on one line (mostly the vertical aligning of the checkbox + text + site selector). I don't believe it's worth inserting so many hacks for something that will not look good visually. If you want to give it a try feel free but I'll leave it as it is. Except that all other points should be fixed now. |
it looks good!
if it's hard to solve then +1 to skip it for now, it was a small feedback and not "required" |
I tried to test all features manually, here is additional feedback:
Current H1 style in UI demo:
(some of these refs #7587) |
@mattab all points should be addressed now. FYI the "disabled fields" where styled (since the forms redesign) but it was lost in a regression -> fixed. For this: I have changed the style to this: |
👍 New points:
|
Regarding this:
This is an unrelated issue that has nothing to do with the redesign. Let's stay on topic and open these in separate tickets. I created #8109 for this. |
👍 If anyone notices a UI element not displayed correctly let us know! As far as I can see, it looks all good now. |
For the record I'll update the UI screenshots now, I forgot to do that yesterday while waiting for the build |
This issue aims to collect small feedback about UI & design:
In admin forms, website selector should be to the right of the label (Got: below the label)What we got does not look ideal imho:
Looks broken on Firefox:
Feel free to add here any other design 'regression' or improvements that should be made before 2.14.0 👍
The text was updated successfully, but these errors were encountered: