-
-
Notifications
You must be signed in to change notification settings - Fork 434
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
feat: match theme-color with primary-color #774
Conversation
Signed-off-by: Kerim Bilgic <[email protected]>
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.
Thank you @BastelKlug, I tested on Windows 11 with Edge browser, working great! 🙂
neat, thanks! |
@@ -19,7 +19,7 @@ module.exports = defineConfig({ | |||
} | |||
}, | |||
pwa: { | |||
themeColor: '#2196F3', | |||
themeColor: '', |
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.
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.
Yes, when I removed the line, it took the default value from the plugin. So I left it blank. Maybe there is another way 🤔
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.
MS Edge in Win11 seems to work fine with the value in... What browser/phone did you notice that you needed to clear that? Might be the case here...
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.
Chrome on Windows 11. Maybe it was a bug with the Vue Cli Hot reloading?
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.
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.
Ah, I just tested with Chrome on Win11 and worked fine... guess this is a Mac thing only then?
Closes: #406
Hi, I think the changes are self-explaining.
Here is a demo video:
theme-color.mp4
Signed-off-by: Kerim Bilgic [email protected]