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

make theme_color of Manifest can be set #1542

Merged
merged 2 commits into from
Sep 10, 2021
Merged

Conversation

bekcpear
Copy link
Contributor

@bekcpear bekcpear commented Sep 7, 2021

The 'theme_color' of Manifest affects the color of status bar
of mobile view, however it's static before. Make it can be set
via filebrowser config set --branding.color "<hex-color-code>".

Signed-off-by: bekcpear i@bitbili.net

The 'theme_color' of Manifest affects the color of status bar
of mobile view, however it's statc before. Make it can be set
via `filebrowser config set --branding.color "<hex-color-code>"`.

Signed-off-by: bekcpear <i@bitbili.net>
@bekcpear bekcpear requested a review from o1egl as a code owner September 7, 2021 07:25
Copy link
Contributor

@ramiresviana ramiresviana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Chrome desktop/mobile ignores the theme-color from dynamic manifest. You have to use template variables on index.html#L19 and index.html#L29.

Thanks @ramiresviana for pointing out the problem:
"Chrome desktop/mobile ignores the theme-color from dynamic manifest."
@o1egl o1egl merged commit 0358e42 into filebrowser:master Sep 10, 2021
@bekcpear bekcpear deleted the theme_color branch September 17, 2021 19:49
rahul-r pushed a commit to rahul-r/filebrowser that referenced this pull request Sep 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants