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

Better update channel selector design #38922

Conversation

Jerome-Herbinet
Copy link
Member

Before

2023-06-21_10-09

After

2023-06-21_10-09_1

Checklist

Copy link
Member

@jancborchardt jancborchardt left a 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?

@Jerome-Herbinet
Copy link
Member Author

Looks nice! But only the right part (the channel itself) should be inside the border, no?

@jancborchardt it's done and looking fine IMO :
2023-06-23_11-17
If it's OK for you, tell me, so I can squash all this.

@Jerome-Herbinet Jerome-Herbinet force-pushed the Jerome-Herbinet-better-update-channel-selector-design branch from 9c45aff to bf43069 Compare June 23, 2023 14:39
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]>
@Jerome-Herbinet Jerome-Herbinet force-pushed the Jerome-Herbinet-better-update-channel-selector-design branch from bf43069 to a1aed1d Compare June 23, 2023 14:39
@Jerome-Herbinet
Copy link
Member Author

@szaimen I let you check if everything is OK for you.

Copy link
Contributor

@szaimen szaimen left a 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

@susnux
Copy link
Contributor

susnux commented Aug 16, 2023

@Jerome-Herbinet can you resolve the conflicts?

@Jerome-Herbinet
Copy link
Member Author

See respawn #39927

@susnux susnux deleted the Jerome-Herbinet-better-update-channel-selector-design branch August 17, 2023 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants