Opened 12 years ago
Closed 11 years ago
#9426 closed bug (fixed)
Change "Enable notifications" button to checkbox. (easy)
Reported by: | mmadia | Owned by: | pulkomandy |
---|---|---|---|
Priority: | normal | Milestone: | R1 |
Component: | Preferences/Notifications | Version: | R1/Development |
Keywords: | Cc: | secris1@… | |
Blocked By: | Blocking: | ||
Platform: | All |
Description
The enable/disable button should really be a checkbox "[_] Enable notifications"
Attachments (3)
Change History (10)
comment:1 by , 11 years ago
Keywords: | easy added |
---|
comment:2 by , 11 years ago
Keywords: | easy removed |
---|---|
Summary: | Change "Enable notifications" button to checkbox. → Change "Enable notifications" button to checkbox. (easy) |
by , 11 years ago
Attachment: | 0001-Changed-Enable-notifications-button-to-checkbox.patch added |
---|
comment:3 by , 11 years ago
patch: | 0 → 1 |
---|
comment:4 by , 11 years ago
comment:6 by , 11 years ago
Cc: | added |
---|
by , 11 years ago
Attachment: | 0001-Changed-notification-button-to-a-checkbox.patch added |
---|
Patch with all changes included.
by , 11 years ago
Attachment: | 0001-Changed-enable-notification-button-to-a-checkb.patch added |
---|
Final patch? Fixes a potential issue with launching the Notification Server twice.
comment:7 by , 11 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Applied with some extra style fixes in hrev46979. Thanks for working on this!
Note:
See TracTickets
for help on using tickets.
There are some more improvements to do: