Closed Bug 1023663 Opened 10 years ago Closed 10 years ago

[User story] Calendar day view should start at 8 AM

Categories

(Firefox OS Graveyard :: Gaia::Calendar, defect)

Product:
Firefox OS Graveyard
Component:
Gaia::Calendar
Platform:
ARM
Gonk (Firefox OS)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(feature-b2g:2.1)

Status:
VERIFIED FIXED
Milestone:
2.1 S2 (15aug)
Project Flags:
feature-b2g 2.1

People

(Reporter: skasetti, Assigned: evanxd)

References

Details

(Whiteboard: [tako][p=7][2.1-feature-qa+])

User Story

As a user I want the day view to start at 8AM so it represents my working day.

Attachments

(2 files, 2 obsolete files)

Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/22352
(deleted), text/x-github-pull-request
mmedeiros
: review+
harly
: ui-review+
Details
[2.1 Calendar] 8AM Day View_v1.1.pdf
(deleted), application/pdf
Details
No description provided.
Attached file [2.1 Calendar] 8AM Day View_Draft.pdf (obsolete) (deleted) —
QA Contact: edchen
Assignee: nobody → evanxd
Whiteboard: [tako]
Target Milestone: --- → 2.1 S1 (1aug)
Attached file [2.1 Calendar] 8AM Day View_v1.0.pdf (obsolete) (deleted) —
Attachment #8448477 - Attachment is obsolete: true
Mentor: jlal
Evan, you may want to take a look at the patch on Bug 805573 to see if that code can be re-used or incorporated into your fix here.
Hi Dylan, Thanks for this. I was investigating the smooth scrolling things.
Currently, the patch is for ui review. It is not ready for code review.
Comment on attachment 8465321 [details] Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/22352 Hi Harly, Could you help to do the ui-review? Thanks.
Attachment #8465321 - Flags: ui-review?(hhsu)
Attachment #8465321 - Flags: ui-review?(hhsu)
Attachment #8465321 - Flags: ui-review?(hhsu)
Attached file [2.1 Calendar] 8AM Day View_v1.1.pdf (deleted) —
Attachment #8463766 - Attachment is obsolete: true
Comment on attachment 8465321 [details] Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/22352 Hi Evan, I have added a new spec to auto scroll to an hour before current time in current day view. Other than that, the patch looks great. Thanks!!
Attachment #8465321 - Flags: ui-review?(hhsu) → ui-review+
Hi Harly, I already updated the patch for the an hour before thing. Thanks for the review.
Depends on: 1047279
Depends on: 1047283
Depends on: 1039983
Hi Harly, I think your meaning is these three bug depend on Bug 1023663.
Blocks: 1039983, 1047279, 1047283
No longer depends on: 1039983, 1047279, 1047283
Whiteboard: [tako] → [tako][p=5]
Comment on attachment 8465321 [details] Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/22352 Hi Miller, Could you help to review the patch? I will add(fix) the related unit/marionette tests after the first round review. Thanks.
Attachment #8465321 - Flags: review?(mmedeiros)
Comment on attachment 8465321 [details] Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/22352 added a few comments about the code
Attachment #8465321 - Flags: review?(mmedeiros)
QA Whiteboard: [2.1-feature-qa+]
Hey guys this was targeted to land in sprint 1, is this moving along? Thanks
Flags: needinfo?(mmedeiros)
Flags: needinfo?(evanxd)
Hi Candice, We're doing code review now, and the ux review is done. I think we could land this in this week.
Flags: needinfo?(evanxd)
Target Milestone: 2.1 S1 (1aug) → 2.1 S2 (15aug)
QA Whiteboard: [2.1-feature-qa+] → [COM=Productivity]
QA Whiteboard: [COM=Productivity] → [COM=Gaia::Calendar]
QA Whiteboard: [COM=Gaia::Calendar] → [COM=Gaia::Calendar][2.1-feature-qa+]
Comment on attachment 8465321 [details] Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/22352 Hi Miller, If the patch[1] looks OK, I will add/fix the unit/marionette tests tomorrow. And then I will send you review the test code again. Thanks. [1] https://github.com/mozilla-b2g/gaia/pull/22352
Attachment #8465321 - Flags: feedback?(mmedeiros)
Evan, the code is looking good! I did not spend too much time looking at the test tho since you said you had to fix/add some things. Thanks!
Flags: needinfo?(mmedeiros)
Hi Harly and Peko, For the scrolling effect, we could use the `ease` effect. How do you think?
Flags: needinfo?(pchen)
Flags: needinfo?(hhsu)
Hi Evan, yes, I agree with you. The `ease` effect is better. thanks!!
Flags: needinfo?(pchen)
Mentor: jlal
Thanks Evan, The ease effect fell much better than the constant speed effect. Thank you for adding it.
Flags: needinfo?(hhsu)
Comment on attachment 8465321 [details] Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/22352 Hi Miller, I updated for the comments. Could you help to review the patch? Thanks.
Attachment #8465321 - Flags: feedback?(mmedeiros) → review?(mmedeiros)
Comment on attachment 8465321 [details] Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/22352 looking good! only thing that annoys me should be fixed by Bug 1047283. good work. PS: it looks like it broke one of integration tests (TEST-UNEXPECTED-FAIL | /builds/slave/test/gaia/apps/calendar/test/marionette/day_view_test.js | day view events longer than 2h click after event end)
Attachment #8465321 - Flags: review?(mmedeiros) → review+
Flags: in-moztrap?(edchen)
QA Whiteboard: [COM=Gaia::Calendar][2.1-feature-qa+] → [COM=Gaia::Calendar]
Whiteboard: [tako][p=5] → [tako][p=5][2.1-feature-qa+]
Hi Miller, Thanks for the review. I already fixed the day_view_test.js failure. For the failures of Gip and Gij[1] are due to Bug 1048953. [1] https://tbpl.mozilla.org/?rev=cca49194eb6dc78724a4daa3677c5e62ec66d01a&tree=Gaia-Try
master: db50c8d84054c0286c3ea355feae32ee4a51c82c
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [tako][p=5][2.1-feature-qa+] → [tako][p=7][2.1-feature-qa+]
Blocks: 1051752
Blocks: 1052252
[Environment] Gaia c8e93dadeaec6a2617b4db8b7d65ba14aa8db378 Gecko https://hg.mozilla.org/mozilla-central/rev/c14e5feadc61 BuildID 20140821160203 Version 34.0a1 ro.build.version.incremental=94 ro.build.date=Tue May 20 09:29:20 CST 2014 [Functional testing Result] 4 out of 4 testcases are passed for the 2014-08-21 Flame.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: