FriendsOfFlarum / nightmode

🌙 Turn off the lights!
https://discuss.flarum.org/d/21492-friendsofflarum-night-mode
MIT License
33 stars 22 forks source link

Provide an option to display the sun or moon icon in solid #73

Closed Yippy closed 1 month ago

Yippy commented 7 months ago

Added a new option for Admin's the ability to change the icon between solid and outline, in my opinion the sun outline looks like a cog. Also for consistency for the website theme, many uses solid icons.

Changes proposed in this pull request: This effects the extension setting with the added new option. Header and dropdown option icon, through-out the website.

Reviewers should focus on: Top right corner icons

Screenshot image

Confirmed

Yippy commented 7 months ago

Noticed an old issue mentioning about the sun looking like a gear icon, though this PR is defaulted for the regular style with the option to change to solid. I'm not sure how to do a composer test, but the PR is ready for review. Fixes: #40

imorland commented 6 months ago

Hey! Thanks for the PR!

I actually think that we should just switch over to using the solid icons here anyway, as these would then match the default icons used across Flarum.

Would you mind making that change for me?

PS, please don't commit the dist files, these will be built automatically by the bot once merged :)

Yippy commented 6 months ago

Hey! Thanks for the PR!

I actually think that we should just switch over to using the solid icons here anyway, as these would then match the default icons used across Flarum.

Would you mind making that change for me?

PS, please don't commit the dist files, these will be built automatically by the bot once merged :)

But wouldn't this effect everyone with this package, as I'm not sure if it's too late and people got use to the outline. Yes the original default theme only use solid icons. Do you want me to:

  1. Remove the toggle and just assume everyone want solid icon
  2. Keep the toggle, but default to solid icon. Providing the people the option to change to outline?
imorland commented 1 month ago

After a settings page refactor, I've reimplemented this in #73