-
-
Notifications
You must be signed in to change notification settings - Fork 357
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
Issues with the top bar when using a theme other then the default (dark and sepia) #978
Comments
Thanks for the report! |
Thank you for the help with the menus, that means that the images for desktop(#978 (comment)) are taken from Firefox 95.0.2 |
I can reproduce this on Vivaldi browser version 5.0.2497.32, installed on my Ubuntu 20.04 machine. This seems to be intentional, is in the light theme bar is opaque, while the Dark and Sepia themes have a transparent bar. This will probably a simple CSS change (if JS doesn't mess with it), if someone can do this or point me to the themes. |
question, should I mark also are there any other tags that should be added to this issue? |
I think this can be solved by adding these lines to .book.color-theme-1 .book-header {
color: #afa70;
background: #f3eacb;
}
.book.color-theme-2 .book-header {
color: #7e888b;
background: #1c1f2b;
} It worked on Firefox and Chromium on my Arch Linux machine. |
This should probably be solved using CSS variables, but this should be good enough, I'll look at it when I have some time |
The CSS variables of themes are written in gitbook-plugin-fontsettings. I found these two lines cause the transparent top bar, but I do not know how gitbook/honkit plugins works, so cannot go any further. |
I think we use the default gitbook themes, which makes it hard to change them. We seem to be using those plugins, which doesn't seem to include the plugin you linked:
EDIT: Looking at the dependencies, Honkit seems to use the (now read-only) plugin you pointed to indeed. |
So from what I can tell this is fixable, am I wrong here, does this mean that I can mark it |
I think we got caught up in the fact that the theme api plugin does not technically have a license attached to it, so we were uncomfortable modifying it. The fonts plugin is apache, so it's fine to fork and modify that. Maybe the best course of action is to fork that one, make the changes and see if setting it as a dependency fixes the issue. |
Bug Report
Description
When on mobile, and the theme is sepia or dark and you scroll up from the top the bar at the top disappears which makes the UI hard to read and interact with
Steps to Reproduce
Steps to reproduce the behavior:
Expected behavior
From what I can tell there is supposed to be be a bar behind the language selector and the A (I have no clue what it is called)
Screenshots
This first screen shot is of the default theme with what seams to be the default and correct behavior
Next is the sepia and dark themes in that order which have issue
Environment
Additional context
I am on IOS and don't have access to Android so I could not test it in those devices; I also haven't had the chance to test on desktop. Also I would like to note that this is my first issue on this project so I am sorry if I made a mistake when typing this up.
For Algorithm Archive Developers
The text was updated successfully, but these errors were encountered: