[Weather app] Manage location screens are implemented differently to visual spec

Bug #1187314 reported by Christina Li on 2013-06-04
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu UX
Medium
Christina Li
Ubuntu Weather App
High
Martin Borho

Bug Description

Manage location differs from Visual Designs. Please align.
-----------------------------------------------------------------

1. Tool bar should say 'Edit' instead of 'Add' for location
2. The experience should be implemented as spec, see link...under 'Manage locations'
http://design.canonical.com/2013/03/app-patterns-applied-weather-key-journeys/

Related branches

Changed in ubuntu-weather-app:
status: New → Confirmed
Changed in ubuntu-weather-app:
assignee: nobody → Martin Borho (martin-borho)
status: Confirmed → In Progress
Changed in ubuntu-weather-app:
status: In Progress → Fix Released
Changed in ubuntu-ux:
status: New → Fix Released
Changed in ubuntu-ux:
status: Fix Released → Triaged
importance: Undecided → Medium
Changed in ubuntu-weather-app:
status: Fix Released → New
status: New → Fix Committed
status: Fix Committed → Fix Released
status: Fix Released → In Progress
Christina Li (christina-li) wrote :

Attached is the patterns to be used for Manage Location

Martin Borho (martin-borho) wrote :

Hi Christina,

thanks for the pattern!

Problem is: you can't open a Page from a Sheet without closing the Sheet and/or pushing a Page on top of the MainPage. Therefore opening a Page from a Sheet would not result in a logical page tree. Part of this is, that the PageStack should always be bound to the MainWindow. It's not possible to create a PageStack inside of a Sheet compoment.

So I think, the pattern should be reconsidered. Perhaps adjusting the design of the current LocationManagerPage would be the better way.

Cheers
Martin

Changed in ubuntu-weather-app:
status: In Progress → Opinion
Changed in ubuntu-ux:
assignee: nobody → Christina Li (christina-li)
Christina Li (christina-li) wrote :

Hi Martin,
Thanks for the feedback.

Based on this, let's remove the page stack pattern and use the sheet pattern throughout the edit location.
When a user is searching for a list of cities- the 'cancel' button will take the user back to the main page.
When a user is searching for a list of cities by tapping to adding a city will automatically take the user back to the main page (without the need to hit done).

let me know if you got other questions.

summary: - Manage location screens are implemented differently to wireframe spec
+ [Weather app] Manage location screens are implemented differently to
+ wireframe spec
Changed in ubuntu-weather-app:
status: Opinion → In Progress
importance: Undecided → High

Fix committed into lp:ubuntu-weather-app at revision 71, scheduled for release in ubuntu-weather-app, milestone alpha-1

Changed in ubuntu-weather-app:
status: In Progress → Fix Committed
John Lea (johnlea) on 2013-08-08
Changed in ubuntu-ux:
status: Triaged → Fix Committed
David Planella (dpm) on 2013-09-03
Changed in ubuntu-weather-app:
status: Fix Committed → Fix Released
Christina Li (christina-li) wrote :

Hi, re-opening this bug as Manage location isn't implemented according to visual design spec yet.

Changed in ubuntu-weather-app:
status: Fix Released → New
summary: [Weather app] Manage location screens are implemented differently to
- wireframe spec
+ visual spec
description: updated
Martin Borho (martin-borho) wrote :
Christina Li (christina-li) wrote :

Hi Martin,
Both- Michal had a look at the current visual implementation yesterday and he has provided a 'Add a city' screen visual design which we need to look at implementing.

Martin Borho (martin-borho) wrote :

Okay, nonetheless we need an updated Sheet component and the visiual design assets before we can move on with implementing the visual design in the "Add a city" sheet.

Cheers
Martin

David Planella (dpm) wrote :

Which elements need still fixing on this bug? It might be worth marking this one as Fix Released and filing new bugs for each part that still needs design or SDK implementation, as otherwise it's difficult to track the work required.

Changed in ubuntu-weather-app:
status: New → Incomplete
Changed in ubuntu-weather-app:
status: Incomplete → In Progress
status: In Progress → Incomplete
Changed in ubuntu-weather-app:
status: Incomplete → Won't Fix
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers