Merge lp:~felix-lawrence/weather-indicator/tweak-location-searching into lp:weather-indicator/3.0
Proposed by
Felix Lawrence
Status: | Merged |
---|---|
Approved by: | Vadim Rutkovsky |
Approved revision: | 238 |
Merged at revision: | 219 |
Proposed branch: | lp:~felix-lawrence/weather-indicator/tweak-location-searching |
Merge into: | lp:weather-indicator/3.0 |
Diff against target: |
24 lines (+2/-1) 2 files modified
bin/indicator-weather (+1/-0) data/ui/Assistant.ui (+1/-1) |
To merge this branch: | bzr merge lp:~felix-lawrence/weather-indicator/tweak-location-searching |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Weather Indicator Team | Pending | ||
Review via email: mp+56066@code.launchpad.net |
Description of the change
This makes searching for locations a bit more intuitive.
- The label text "Please input a location in the combobox below:" has been changed to "Please search our database for your location:", which warns people that they are going to have to click the search button.
- When the Search button is clicked and the results are found, the menu of results now appears automatically. Hopefully this will reduce the need for a progress indicator (bug #748116), since it is now much clearer that the operation is done.
Vadim: is it more convenient for you if I send you code in this way, or should I file a bug and attach a patch file?
To post a comment you must log in.