[go: nahoru, domu]

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

Allow auto dark mode and forcing theme #153

Merged
merged 3 commits into from
May 12, 2024
Merged

Conversation

chrisrhymes
Copy link
Owner

Resolves #151, resolves #152

  • Fixes bug with fixed navbar and light theme being forced.
  • Allows auto dark mode detection or forcing a theme
  • Add additional information to upgrade guide and improved docs based on feedback

@chrisrhymes chrisrhymes added bug Something isn't working enhancement New feature or request labels May 7, 2024
@@ -1,5 +1,8 @@
<div x-data="{visible: true}" markdown="0">
<div class="notification {{ include.status | default: 'is-warning' }}" x-show.transition.duration.300ms="visible">
<div
Copy link
Owner Author

Choose a reason for hiding this comment

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

Add margin to top and bottom of notifications

@GruberMarkus
Copy link

I kindly ask for the following change to #153: Don't make force_theme: light the default, but let bulma.io decide if light or dark mode should be used based on the user's system and browser settings (#force_theme: light).

@@ -11,6 +11,13 @@ show_sidebar: false

Bulma Clean Theme uses the Bulma frontend framework. Check out the [Bulma docs](https://bulma.io/documentation/) for more information.

## Bulma Themes

Bulma v1 has the concept of [themes](https://bulma.io/documentation/features/themes/) and auto detects the browsers choice for dark or light theme.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Bulma v1 has the concept of [themes](https://bulma.io/documentation/features/themes/) and auto detects the browsers choice for dark or light theme.
Bulma v1 has the concept of [themes](https://bulma.io/documentation/features/themes/) and auto detects the browser's choice for dark or light theme.


```yaml
# With GitHub Pages Gem
remote_theme: chrisrhymes/bulma-clean-theme:v0.14.0
Copy link
Contributor

Choose a reason for hiding this comment

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

This is much clearer, thank you! I didn't know that you could pin the version of a remote theme. That looks like a simple solution to start with.


```yaml
# _config.yml
remote_theme: chrisrhymes/bulma-clean-theme@v1.0.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Would you like this to be consistent with the README and installation pages? Both worked in my testing, so this isn't wrong.

Suggested change
remote_theme: chrisrhymes/bulma-clean-theme@v1.0.0
remote_theme: chrisrhymes/bulma-clean-theme:v1.0.0

@chrisrhymes
Copy link
Owner Author

I kindly ask for the following change to #153: Don't make force_theme: light the default, but let bulma.io decide if light or dark mode should be used based on the user's system and browser settings (#force_theme: light).

Thanks for your feedback. I have been thinking about this and decided to keep it as force_light by default as this is similar behaviour to how the previous version looked. You are free to set whatever value you like in your site's own _config.yml.

@chrisrhymes chrisrhymes merged commit 079cde9 into master May 12, 2024
@chrisrhymes chrisrhymes deleted the fix/auto-dark-mode branch May 12, 2024 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
3 participants