Possible to add blank or non-existent locations
Bug #740945 reported by
Matthew Paul Thomas
This bug affects 2 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Indicator Date and Time |
Fix Released
|
Undecided
|
Unassigned | ||
indicator-datetime (Ubuntu) |
Fix Released
|
Low
|
Unassigned |
Bug Description
Binary package hint: indicator-datetime
indicator-datetime 0.1.98-0ubuntu1, Ubuntu Natty
1. From the clock menu, choose "Time & Date Settings...".
2. Check "Time in other locations" (if it isn't checked already).
3. Click "Choose Locations...".
4. Click Add, and press Enter.
5. Click Add, type "asdf", and press Enter.
What happens:
4. A blank location is added.
5. An "asdf" location is added.
What should happen:
4. No location is added.
5. No location is added.
Related branches
lp:~mterry/indicator-datetime/handle-enter
- Indicator Applet Developers: Pending requested
-
Diff: 190 lines (+107/-5)3 files modifiedsrc/datetime-prefs-locations.c (+0/-1)
src/datetime-prefs.c (+0/-2)
src/timezone-completion.c (+107/-2)
lp:~mterry/indicator-datetime/error-icon
- Indicator Applet Developers: Pending requested
-
Diff: 96 lines (+35/-7)2 files modifieddata/datetime-dialog.ui (+4/-0)
src/datetime-prefs-locations.c (+31/-7)
lp:~mterry/indicator-datetime/more-error-icons
- Indicator Applet Developers: Pending requested
-
Diff: 133 lines (+70/-6)2 files modifiedsrc/datetime-prefs-locations.c (+24/-0)
src/datetime-prefs.c (+46/-6)
Changed in indicator-datetime (Ubuntu): | |
status: | New → Confirmed |
Changed in indicator-datetime (Ubuntu): | |
importance: | Undecided → Low |
Changed in indicator-datetime (Ubuntu): | |
assignee: | nobody → Michael Terry (mterry) |
status: | Confirmed → In Progress |
Changed in indicator-datetime: | |
milestone: | none → 0.2.1 |
status: | New → Fix Committed |
Changed in indicator-datetime: | |
status: | Fix Committed → Fix Released |
Changed in indicator-datetime (Ubuntu): | |
assignee: | Michael Terry (mterry) → nobody |
status: | In Progress → Fix Released |
To post a comment you must log in.
I have an attached branch for the "pressing enter" part of this. The "focus out" part is still a work in progress.