Volume notification is based on previous volume; should be based on the new volume
Bug #748831 reported by
Dylan McCall
This bug affects 3 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
The Sound Menu |
Fix Released
|
Low
|
Conor Curran | ||
indicator-sound (Ubuntu) |
Fix Released
|
Undecided
|
Conor Curran |
Bug Description
Binary package hint: indicator-sound
When I change the volume with the mouse wheel, a notification bubble now appears by default. (This is toggled with the show-notify-
This is easy to see by slowly reducing the volume to 0, then scrolling up very gently with the mouse.
Related branches
lp:~dylanmccall/indicator-sound/notifications-use-new-volume
- Conor Curran (community): Approve
-
Diff: 179 lines (+88/-21)5 files modifiedsrc/Makefile.am (+4/-0)
src/device.c (+2/-15)
src/sound-state-manager.c (+8/-6)
src/sound-state.c (+43/-0)
src/sound-state.h (+31/-0)
Changed in indicator-sound: | |
status: | New → Triaged |
importance: | Undecided → Low |
assignee: | nobody → Conor Curran (cjcurran) |
Changed in indicator-sound (Ubuntu): | |
status: | New → Confirmed |
assignee: | nobody → Conor Curran (cjcurran) |
Changed in indicator-sound: | |
milestone: | none → 0.7.0 |
Changed in indicator-sound: | |
status: | In Progress → Fix Committed |
Changed in indicator-sound (Ubuntu): | |
status: | Confirmed → In Progress |
Changed in indicator-sound: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Thanks Dylan, I will review and merge this after the Natty release. From the start of last week I have been away from work and will be for the next three weeks at least. Considering that bug is not a show stopper and the code change is pretty significant I was not prepared to merge this at this late point in the cycle. I hope you understand. If bugs were to emerge then I would need to work weekends in order to fix them :)
Will get this in once the O cycle begins ...