Merge lp://staging/~costales/unav/0.59-new_fixes into lp://staging/unav

Proposed by costales
Status: Merged
Merged at revision: 56
Proposed branch: lp://staging/~costales/unav/0.59-new_fixes
Merge into: lp://staging/unav
Diff against target: 366 lines (+44/-87)
8 files modified
app-armor.json (+0/-1)
nav/class/UI.js (+1/-1)
nav/index.html (+2/-20)
qml/Coordinate.qml (+4/-8)
qml/Favorites.qml (+4/-2)
qml/Location.qml (+6/-2)
qml/Main.qml (+23/-51)
qml/PoiListPage.qml (+4/-2)
To merge this branch: bzr merge lp://staging/~costales/unav/0.59-new_fixes
Reviewer Review Type Date Requested Status
Nekhelesh Ramananthan Approve
Review via email: mp+292422@code.staging.launchpad.net

Description of the change

- Fixed dark purple in navigation panel.
- Fixed: Set a route. Search > Click on any Location entry = Bug: Center button is disabled.
- Fixed: open uNav > Click on map (you will see one popup) > click on a favorite or search > bug: popup will be the previous.
- Fixed: open uNav > Set route > Click on map (you'll see the popup) > click on current pos, destination or origin > Bug: popup will not update the text.

To post a comment you must log in.
59. By costales

Bug: Popup not updated

60. By costales

Revert testing code commented

Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

Please update screenshot in welcome wizard of the navigation panel where you changed orange to blue.

review: Needs Fixing
61. By costales

Window size for convergence in tablet

62. By costales

Restored orange in indication (blue is soft and in so many places

63. By costales

Window size for convergence in tablet

64. By costales

Removed AGPS popup

65. By costales

Fixed click on pos, destination, origin with popup

66. By costales

Updated testing code

Revision history for this message
costales (costales) wrote :

@Nekhelesh: Restored to orange again. The blue is so light and it's a so blue in the UI. Not nice. The orange is the blue's complementary, then good color.

Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

I didnt find any regressions. Looks good to me. I am not fully sure about the AGPS dialog removal. But I don't much about the location services to comment on that. With that I am approving this MP.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches