Replace dialog with combotext

Bug #1483534 reported by Fabio Zaramella
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Hourglass
Fix Released
Undecided
Sam Thomas

Bug Description

Sorry but there is no need to spawn another child window to select in when to repeat an alarm.
The combotext was a better choice for that, and it was more understandable.
I meant to add just an Every Day option in my previous bug report.

Revision history for this message
Sam Thomas (sgpthomas) wrote :

The reason I replaced it with a dialog was so that a person could have an alarm repeat in any possible combination of days. Ideally, I would like a combobox that would allow multiple selection. I spent a little time trying to write a custom one, but had no luck. Maybe I'll switch it back to the combobox for now until I can find a better solution.

Changed in hourglass:
milestone: none → 1.0
Revision history for this message
Fabio Zaramella (fabiozaramella) wrote :

I like the idea to set different combinations but i'm still quite diffident about the new dialog. Well, you can leave it at this point.

Revision history for this message
Sam Thomas (sgpthomas) wrote :

I have switched the dialog to a popover. I think that is sort of the best of both worlds. For know I am going to mark this bug as Fix Committed. Let me know what you think.

Changed in hourglass:
status: New → Fix Committed
assignee: nobody → Sam Thomas (sgpthomas)
Sam Thomas (sgpthomas)
Changed in hourglass:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.