Ubuntu Calendar App

[calendar app] event time is wrong when creating a new event

Bug #1193090 reported by Carla Sella on 2013-06-20

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	Ubuntu Calendar App	Fix Released	Undecided	Riccardo Padovani
	Ubuntu UX	Fix Released	Medium	Unassigned

Bug Description

When you create a new event in calendar app, the time of the event is wrong:
- Select "New Event" from the toolbar
- Insert something in all the fields a part the hour and minute ones
- in the hour field insert 22
. in the minute field insert 17 (or anything different from 0)
- save the event
you will see that the event time is displayed in the "Events " list as 22:00 instead of 22:17.

See original description

Tags:

Related branches

lp:~rpadovani/ubuntu-calendar-app/1193090

Merged into lp:ubuntu-calendar-app at revision 68

Ubuntu Phone Apps Jenkins Bot: Needs Fixing (continuous-integration) on 2013-07-12

Riccardo Padovani (community): Approve on 2013-07-11

Olivier Tilloy (community): Approve on 2013-07-11

lp:~nskaggs/ubuntu-calendar-app/autopilot-fixes-for-newevent (Merged)

Carla Sella (carla-sella) on 2013-06-20

summary:	- [calendar app] time is wrong when creating a new event + [calendar app] event time is wrong when creating a new event
description:	updated

Revision history for this message

Riccardo Padovani (rpadovani) wrote on 2013-06-20:

Screenshot from 2013-06-21 00:44:26.png Edit (74.4 KiB, image/png)

Looking at code (I'm not involved in the development of the app) there is no minute field: right field is for the end hour.
So, if you insert a new event with 22 and 17 you create an event that start at 10pm and finish at 5pm (see attachment).

So we have 3 bugs:
a - design bug: it's not simply to understand that right field is for end hour
b - logical bug: it's impossible that an event finish before its start.
c - implementation bug: we need a method to set minutes

c is very simply, just to modify [1] and [2] from Qt.formatDateTime(defaultDate,"hh") to Qt.formatDateTime(defaultDate,"hh:mm") but before making a patch rather wait a developer to know the causes of the choices

[1]http://bazaar.launchpad.net/~ubuntu-calendar-dev/ubuntu-calendar-app/trunk/view/head:/NewEvent.qml#L75
[2]http://bazaar.launchpad.net/~ubuntu-calendar-dev/ubuntu-calendar-app/trunk/view/head:/NewEvent.qml#L91

Regards,

tags:

added: needs-design

Riccardo Padovani (rpadovani) on 2013-06-22

Changed in ubuntu-calendar-app:
status:	New → Confirmed
assignee:	nobody → Riccardo Padovani (rpadovani)
Changed in ubuntu-ux:
status:	New → Fix Released

Revision history for this message

Riccardo Padovani (rpadovani) wrote on 2013-06-22:

I modified the UI for a new event following this article[1], adding the label 'to:'
I also modified the input text to set minutes, in addition to hours.

[1]http://design.canonical.com/2013/06/solving-multiple-calendar-views/

Changed in ubuntu-calendar-app:
status:	Confirmed → Fix Committed

Olivier Tilloy (osomon) on 2013-09-02

Changed in ubuntu-calendar-app:
status:	Fix Committed → Fix Released

Revision history for this message

John Lea (johnlea) wrote on 2013-09-17:

@rpadovani; hi, please do not change the status of bugs in the "ubuntu-ux" project, as we use this project to track design status and final design QA. For example, changing status from Fix Committed → Fix Released in ubuntu-ux means that a designer has tested the implementation of the bug on the phone and has confirmed that implementation matches the change request.

Full details of what status means for ubuntu-ux bugs is available at https://docs.google.com/a/canonical.com/document/d/1PcNRQZFbfmmQEvxu4SZHO3GZ9Zaetg9p2DOmRwOUbS0/edit#heading=h.yt7pgvf4mg7m

Reverting ubuntu-ux status back to Fix Committed so that designers know that the implementation is ready to test.

Changed in ubuntu-ux:
status:	Fix Released → Fix Committed

Matthew Paul Thomas (mpt) on 2014-09-12

Changed in ubuntu-ux:
assignee:	nobody → Benjamin Keyser (bjkeyser)

Olga Kemmet (olga-kemmet) on 2014-12-09

Changed in ubuntu-ux:
assignee:	Benjamin Keyser (bjkeyser) → Giorgio Venturi (giorgio-venturi)
importance:	Undecided → Medium
status:	Fix Committed → Triaged