Skip to content

Addition of a user-selectable dark-mode option #2022

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

Merged
merged 6 commits into from
Apr 11, 2020
Merged

Conversation

ssddanbrown
Copy link
Member

@ssddanbrown ssddanbrown commented Apr 10, 2020

  • Most elements done, but still need to do editors, tables and final pass.
  • Toggled only by quick js check at the moment, checking via css media query. Need to make into user-preference toggle.
  • Say thanks to domainzero

For #1234

- Most elements done, but still need to do editors, tables and final
pass.
- Toggled only by quick js check at the moment, checking via css media
query. Need to make into user-preference toggle.

For #1234
@ssddanbrown ssddanbrown self-assigned this Apr 10, 2020
@ssddanbrown ssddanbrown changed the title Started work on supporting a dark-mode Addition of a user-selectable dark-mode option Apr 10, 2020
@ssddanbrown
Copy link
Member Author

bookstack local_ (1)

- Updated editor, and other area, styles to look okay in dark mode.
- Used tinyMCE theme generator to create dark mode theme.
- Updated tinymce to latest 4x version.
Generally seems to cause issues when secure images are in use.
Was added during laravel upgrade but laravel does not use this directly
for its web middleware anyway.
- Also fixed some remaining areas which needed dark mode support.
@ssddanbrown ssddanbrown added this to the v0.29.0 milestone Apr 11, 2020
@ssddanbrown ssddanbrown merged commit b80b6ed into master Apr 11, 2020
@ssddanbrown ssddanbrown deleted the dark-mode branch April 11, 2020 19:52
@ssddanbrown ssddanbrown mentioned this pull request Apr 11, 2020
@over-soul
Copy link

@ssddanbrown, is there a way to let guest users select dark mode?

@ssddanbrown
Copy link
Member Author

ssddanbrown commented Apr 14, 2020

@over-soul Should be an option on the home screen. See above the right-most "Recent Activity" card here:
https://demo.bookstackapp.com/

@whentotrade
Copy link

Would it be possible to also place this button to switch between dark/light mode on the books, shelf, chapters page sites, e.g. as another item in the "Actions" section below "Export"? This would make is possible to change theme on book/page sites also an not only on the main home screen.

@ssddanbrown
Copy link
Member Author

@whentotrade Dark mode does not really relate to the items that the existing actions are centered around so I'd prefer not to add it there.

Non guest users can toggle dark mode from any page via the top-right profile menu.

@whentotrade
Copy link

whentotrade commented May 13, 2020

Ok, thanks. Then maybe just on the /books page. As there the "dark/light mode" open is missing when you click on the "Books" tab from the tab. While when you choose "books" as the standard homepage, the switch button is there in the overview.

This is the "Actions area" when setting "books" as the standard homepage in the settings (with switch button):
grafik

Where as this is the "Action area" when clicking the main "Books" tab on the top bar (without switch option)
grafik

Then main contant is the same - just the landing page selected as "books" in the settings shows the switch button while when clicking the "books" main tab the option is not visible.

Just a proposal and observation.

Thanks for the great work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants