-
Notifications
You must be signed in to change notification settings - Fork 90
feat(@desktop/browser): Improve incognito mode color scheme #19396
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
Conversation
Jenkins BuildsClick to see older builds (75)
|
8d77b80 to
b0ca456
Compare
b0ca456 to
948696f
Compare
|
I wonder how the colors look like when in full Privacy mode + this Web incognito mode ;) |
948696f to
59afc78
Compare
59afc78 to
f2b74ff
Compare
caybro
left a comment
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.
Colors LGTM but...
f2b74ff to
9404f5c
Compare
caybro
left a comment
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.
Some more remarks, sorry :)
9404f5c to
5bd0686
Compare
5bd0686 to
4b1a11f
Compare
caybro
left a comment
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.
👌
fixes #19256
What does the PR do
implements new incognito mode colors defined in design https://www.figma.com/design/pJgiysu3rw8XvL4wS2Us7W/DS?node-id=3193-73460&m=dev
Affected areas
Browser Header
Architecture compliance
My PR is consistent with this document: QML Architecture Guidelines
Screencapture of the functionality
Impact on end user
How to test
Risk