Merge lp://qastaging/~gary-wzl77/ubuntu-calendar-app/show-week-number into lp://qastaging/ubuntu-calendar-app

Proposed by Gary.Wang
Status: Merged
Approved by: Alan Pope 🍺🐧🐱 πŸ¦„
Approved revision: 726
Merged at revision: 726
Proposed branch: lp://qastaging/~gary-wzl77/ubuntu-calendar-app/show-week-number
Merge into: lp://qastaging/ubuntu-calendar-app
Diff against target: 543 lines (+238/-30)
12 files modified
AgendaView.qml (+2/-1)
AllDayEventComponent.qml (+3/-0)
DayView.qml (+2/-1)
EventActions.qml (+11/-0)
MonthComponent.qml (+86/-1)
MonthView.qml (+4/-1)
Settings.qml (+89/-0)
WeekView.qml (+2/-1)
YearView.qml (+2/-1)
YearViewDelegate.qml (+1/-1)
calendar.qml (+4/-3)
po/com.ubuntu.calendar.pot (+32/-20)
To merge this branch: bzr merge lp://qastaging/~gary-wzl77/ubuntu-calendar-app/show-week-number
Reviewer Review Type Date Requested Status
Alan Pope 🍺🐧🐱 πŸ¦„ (community) Approve
Jenkins Bot continuous-integration Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Nicholas Skaggs (community) Needs Fixing
Mihir Soni Pending
Review via email: mp+264153@code.qastaging.launchpad.net

Commit message

Show week number in monthly and yearly view.

Description of the change

Show week number in monthly and yearly view.
1.Navigate to day view when clicking weeknum in monthly view
2.Navigate to monthly view when clicking weeknum in yearly view

To post a comment you must log in.
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Alan Pope 🍺🐧🐱 πŸ¦„ (popey) wrote :

Not sure about this. We need design to take a look.

http://people.canonical.com/~alan/year.png
http://people.canonical.com/~alan/month.png

Seems cluttered.

Revision history for this message
Gary.Wang (gary-wzl77) wrote :

Add screenshots of monthly view with UI-fined based on feedback from UX team.
Add setting page in latest repo as well.

https://drive.google.com/open?id=0B2H9ECPSSfqIN1AxRGVfMndDYlk
https://drive.google.com/open?id=0B2H9ECPSSfqIc0ZqT2lXOEhTX3M

Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Mihir Soni (mihirsoni) wrote :

Popey,

Any updates from design team regarding this ?

Revision history for this message
Gary.Wang (gary-wzl77) wrote :

Got feedback from UX-design team that
"
This solution works nicely and I think we should proceed with it.
...
"

Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :

FAILED: Autolanding.
More details in the following jenkins job:
http://91.189.93.70:8080/job/ubuntu-calendar-app-autolanding/1405/
Executed test runs:
    FAILURE: http://91.189.93.70:8080/job/ubuntu-calendar-app-vivid-amd64-autolanding/978/console

review: Needs Fixing (continuous-integration)
Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

Text conflict in po/com.ubuntu.calendar.pot
1 conflicts encountered.

review: Needs Fixing
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
710. By Kunal Parmar

resolves Bug #1473679
. Fixes: https://bugs.launchpad.net/bugs/1473679.

Approved by Alan Pope ξƒΏ, Ubuntu Phone Apps Jenkins Bot.

711. By Kunal Parmar

resolves Bug #1479533 . Fixes: https://bugs.launchpad.net/bugs/1479533.

Approved by Alan Pope ξƒΏ, Ubuntu Phone Apps Jenkins Bot.

712. By Launchpad Translations on behalf of ubuntu-calendar-dev

Launchpad automatic translations update.

713. By Launchpad Translations on behalf of ubuntu-calendar-dev

Launchpad automatic translations update.

714. By Launchpad Translations on behalf of ubuntu-calendar-dev

Launchpad automatic translations update.

715. By Kunal Parmar

resolves Bug #1475747 . Fixes: https://bugs.launchpad.net/bugs/1475747.

Approved by Mihir Soni, Ubuntu Phone Apps Jenkins Bot.

716. By Launchpad Translations on behalf of ubuntu-calendar-dev

Launchpad automatic translations update.

717. By Launchpad Translations on behalf of ubuntu-calendar-dev

Launchpad automatic translations update.

Revision history for this message
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
718. By Launchpad Translations on behalf of ubuntu-calendar-dev

Launchpad automatic translations update.

719. By Kunal Parmar

Resolves Bug #1494394 . Fixes: https://bugs.launchpad.net/bugs/1494394.

Approved by Alan Pope ξƒΏ, Jenkins Bot, Ubuntu Phone Apps Jenkins Bot.

720. By Launchpad Translations on behalf of ubuntu-calendar-dev

Launchpad automatic translations update.

721. By Gary.Wang

Tab change will cause dynamic creation(Qt.ResolvedUrl) for qml object back and forth.Using Qt.CreateComponent for dynamic creation for the first time and reuse existing obj without re-creating. Fixes: https://bugs.launchpad.net/bugs/1309263.

Approved by Ubuntu Phone Apps Jenkins Bot, Jenkins Bot, Alan Pope ξƒΏ.

722. By Launchpad Translations on behalf of ubuntu-calendar-dev

Launchpad automatic translations update.

723. By Launchpad Translations on behalf of ubuntu-calendar-dev

Launchpad automatic translations update.

724. By Launchpad Translations on behalf of ubuntu-calendar-dev

Launchpad automatic translations update.

725. By Niklas Wenzel

Remove unnecessary i18n.tr() calls.

Approved by Mihir Soni, Jenkins Bot.

726. By Gary.Wang

Replace calendarStorage with native persistent application settings

Revision history for this message
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Alan Pope 🍺🐧🐱 πŸ¦„ (popey) wrote :

LGTM

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

to status/vote changes: