[Weather app] Settings should look like this
Bug #1197364 reported by
Christina Li
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Ubuntu UX |
Fix Committed
|
High
|
Christina Li | ||
| Ubuntu Weather App |
Fix Released
|
High
|
Martin Borho | ||
Bug Description
1. Temperature units should be used- listing out Celsius or Fahrenheit
2. Use option selector for the selection (will be available in the SDK; option selector details- http://
3. Sheets should be used for settings
See attached wireframe.
Related branches
lp:~martin-borho/ubuntu-weather-app/settings-in-a-sheet
- David Planella: Approve
- Günter Schwann (community): Approve
- Ubuntu Phone Apps Jenkins Bot: Approve (continuous-integration)
- Raúl Yeguas: Pending requested
-
Diff: 378 lines (+182/-115)5 files modifiedcomponents/LocationTab.qml (+20/-2)
components/SettingsPage.qml (+0/-80)
components/SettingsSheet.qml (+63/-0)
tests/autopilot/ubuntu_weather_app/tests/test_settings.py (+77/-28)
ubuntu-weather-app.qml (+22/-5)
Revision history for this message
| Christina Li (christina-li) wrote | #1 |
| Changed in ubuntu-ux: | |
| status: | New → Triaged |
| importance: | Undecided → High |
| assignee: | nobody → Christina Li (christina-li) |
| Changed in ubuntu-weather-app: | |
| assignee: | nobody → Martin Borho (martin-borho) |
| status: | New → Confirmed |
| importance: | Undecided → High |
| Changed in ubuntu-weather-app: | |
| status: | Confirmed → In Progress |
Revision history for this message
| Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote | #2 |
| Changed in ubuntu-weather-app: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| David Planella (dpm) wrote | #3 |
| Changed in ubuntu-weather-app: | |
| status: | Fix Committed → Fix Released |
| Changed in ubuntu-ux: | |
| status: | Triaged → Fix Committed |
To post a comment you must log in.
Fix committed into lp:ubuntu-weather-app at revision 63, scheduled for release in ubuntu-weather-app, milestone alpha-1