Re-enable Trayicon blinking with Snore #328
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
With the change from PR #139 the update to the new Snore Notification Backend modified the behaviour of loading the SystrayNotificationBackend if Snore is enabled.
See: 79ef34a#diff-cfff7c7c664d451740ea7bda9372fc10L83
Although it makes sense to not show a message bubble if snore is enabled, the tray icon animation is handled by SystrayNotificationBackend too.
This might have been overlooked because the tray animation is not en/disabled in the notification settings:
![https://gyazo.com/68b1aa6d8dd93c5e42f67afb3eb6e7e6](https://camo.githubusercontent.com/283868d7f9fce1dbfe1c792ff41ebb715d2c79c3b53d57998286bc674db76ea8/68747470733a2f2f692e6779617a6f2e636f6d2f36386231616136643864643933633565343266363761666233656236653765362e706e67)
With this PR the animation works again but the message bubble has to be disabled manually again:
![https://gyazo.com/17bf819e32f3a7df8a5c9309210097f8](https://camo.githubusercontent.com/5fb5add60d2fe3e1e4e9b42ec96cfb378f4670fe9be2a628399cf4bf216b985f/68747470733a2f2f692e6779617a6f2e636f6d2f31376266383139653332663361376466386135633933303932313030393766382e706e67)
This is how it was before …
It might make sense to auto toggle off the bubble when snore gets enabled, but i didn't find an easy way to do that. (And I'm not completely sure if that's something needed/wanted)
Now https://bugs.quassel-irc.org/issues/1383 should be fixed … a second time …But … Issue 13830 may still not be fixed completely …