-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Better update channel selector design #38922
Better update channel selector design #38922
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks nice! But only the right part (the channel itself) should be inside the border, no?
@jancborchardt it's done and looking fine IMO : |
9c45aff
to
bf43069
Compare
Signed-off-by: Jérôme Herbinet <[email protected]> npm run build Signed-off-by: Jérôme Herbinet <[email protected]> Better update channel selector design Signed-off-by: Jérôme Herbinet <[email protected]> npm run build Signed-off-by: Jérôme Herbinet <[email protected]> Following @jancborchardt suggestion Signed-off-by: Jérôme Herbinet <[email protected]> npm run build Signed-off-by: Jérôme Herbinet <[email protected]> Co-Authored-By: Simon L. <[email protected]>
bf43069
to
a1aed1d
Compare
@szaimen I let you check if everything is OK for you. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested and seems to look and work fine
@Jerome-Herbinet can you resolve the conflicts? |
See respawn #39927 |
Before
After
Checklist