Ubuntu Calendar App

[Calendar] Display Calendar Weeks in yearly and monthly view

Bug #1464407 reported by Johannes Kempf
20
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Ubuntu Calendar App
Fix Committed
Undecided
Gary.Wang
Ubuntu UX
Fix Committed
Low
James Mulholland

Bug Description

Only the weekly view displays the Calendar Week.
In yearly and monthly view an additional column should visualize the according Calendar Week for each row (=week).

--- --- --- ---
UX Comment:

This solution works nicely and I think we should proceed with it.
Three caveats to this recommendation:

1. That the setting for week numbers (in everything but week & day views) be off by default.
2. We use a two letter abbreviation for 'calendar week' (cw, kw, etc) as discussed in comments.
3. We revisit the layout/padding during the planned visual design refresh.

See original description
Tags: needs-design

Related branches

lp://staging/~gary-wzl77/ubuntu-calendar-app/show-week-number
Alan Pope 🍺🐧🐱 🦄 (community): Approve
Jenkins Bot: Approve (continuous-integration)
Ubuntu Phone Apps Jenkins Bot: Approve (continuous-integration)
Nicholas Skaggs (community): Needs Fixing
Mihir Soni: Pending requested
Diff: 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)
Revision history for this message
Alan Pope 🍺🐧🐱 🦄 (popey) wrote :

While I agree that would be a useful addition, it feels to me like the year and month views would become somewhat cluttered if those are added.

Revision history for this message
Johannes Kempf (johanneskempf) wrote :

Of corse, the Calendar Week numbers needs to be displayed different to the day's number.
E.g. another text color or a shaded background color.

Alan Pope 🍺🐧🐱 🦄 (popey)
tags: added: needs-design
Magdalena Mirowicz (magdalena-mirowicz)
summary: - Display Calendar Weeks in yearly and monthly view
+ [Calendar] Display Calendar Weeks in yearly and monthly view
Magdalena Mirowicz (magdalena-mirowicz)
Changed in ubuntu-ux:
assignee: nobody → James Mulholland (jamesjosephmulholland)
importance: Undecided → Low
status: New → Triaged
Revision history for this message
Gary.Wang (gary-wzl77) wrote :

Basically, the common way to display week number is placing week number column in the left side next to first day of week.
Seems there is no much space to display icon,especially in yearly view.So I have a quick fixing to only display text in monthly view and yearly view.
For monthly view: Wk+number
For yearly view: W+number (since there is no enough space to put 4 characters at most,such as Wk31, so I only display one leading characters).
Plz check screenshots for details.

PS: Also I think there might be a way to disable weeknumber display somehow, so a checkbox for this in settings page can be considered.

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

Enclosed Plz find the Montly view

Revision history for this message
Bernard Banko (beernarrd) wrote :

Monthly view feedback: Put CW (aka calendar week) in the row of day names (next to Sun) and keep only the numbers in the first column - so you can save some width (bold or some other colour destiction should avoid the column to be mistaken for the date).
Good idea to make it optional in settings.

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

Thanks eernarrd for your feedback.
Enclosed Plz find latest screenshot of monthly view and settings page as well.
For yearly view, it remains the same.

Revision history for this message
Gary.Wang (gary-wzl77) wrote :
Gary.Wang (gary-wzl77)
Changed in ubuntu-calendar-app:
assignee: nobody → Gary.Wang (gary-wzl77)
status: New → In Progress
Revision history for this message
Johannes Kempf (johanneskempf) wrote :

Feedback to the latest proposal on monthly view:
The german translation for "week" is "Woche". This might not fit into the available space.
Here I would propose to use the common abbreviation "KW" (= CW; calendar week). It is used also in the weekly and daily view.
The abbreviation might fit in the yearly view as well.

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

Clear, but a little concern that
If we use common abbreviation "W" in yearly view, user may mix up with W(ednesday) in day label Row.
My suggestion is that we use "Wk" abbreviation for yearly view & monthly view to keep the same UI layout.
Enclosed Plz find the screenshot.
i18n is needed for sure since we don't have such a msgid "Wk"

Revision history for this message
Gary.Wang (gary-wzl77) wrote :
James Mulholland (jamesmulholland)
description: updated
Changed in ubuntu-ux:
status: Triaged → Fix Committed
Revision history for this message
Jenkins Bot (ubuntu-core-apps-jenkins-bot) wrote :

Fix committed into lp:ubuntu-calendar-app at revision 726, scheduled for release in ubuntu-calendar-app, milestone rtm14

Changed in ubuntu-calendar-app:
status: In Progress → Fix Committed
Revision history for this message
Bernard Banko (beernarrd) wrote :

Thanks.
The fix is not complete; as pointed out in #9, the i18n is needed to make "Wk" translatable. (is not in ota-9)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.