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

Notifications@cinnamon.org : applet improvement #12234

Closed
wants to merge 6 commits into from

Conversation

Hilyxx
Copy link

@Hilyxx Hilyxx commented Jun 14, 2024

Hello

When users enable "Do not disturb" mod, icon isn't updated on the panel.
This commit improves notifications applet with these changes :

  • Add a new icon to panel when "Do not disturb" mod is enabled
  • New menu label when "Do not disturb" mod is enabled
  • Add a toggle switch to easily enable/disable notifications
  • Tooltip now displays notification count
  • Add an option to hide "do not disturb" icon on panel when show empty tray option is enabled (@rcalixte request)
Screencast.2024-06-14.04.42.26.webm

- Add a new icon to panel when "Do not disturb" mod is enabled
- New menu label when  "Do not disturb" mod is enabled
- Add a toggle switch to easily enable/disable notifications
- Add an option to show/hide "Do not disturb" icon (when show empty tray option is disabled)
Add new icon for "Do not disturb" mod
Add an option to keep "do not disturb" icon on panel when show empty tray option is disabled
@rcalixte
Copy link
Member

@Hilyxx Have you considered breaking this out into a separate applet? As a separate Spice, you would have much more flexibility for an implementation.

@Hilyxx
Copy link
Author

Hilyxx commented Jun 20, 2024

Hello @rcalixte
Will this applet be approved if I submit it as a separate applet ?

@rcalixte
Copy link
Member

Hello @rcalixte Will this applet be approved if I submit it as a separate applet ?

I don't see why not. It seems like the perfect solution actually. Similar to CinnaMenu as an alternative menu applet or the few window list ones. Having it separate means that you would have much more freedom with it and also fill a potential need if there are users that prefer an alternative.

@Hilyxx
Copy link
Author

Hilyxx commented Jun 20, 2024

Ok, thanks @rcalixte, I will do that.

@Hilyxx Hilyxx closed this Jun 21, 2024
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.

2 participants