835777 - [Calendar. UX] [Activity Indicator] Wrong activity bar indicating calendar sync

Status: RESOLVED DUPLICATE of bug 933218

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

Description

[Victoria Gerchinhoren [:vicky]]

Attachment: Phone screen capture. Shows wrong activity bar

The current orange activity bar is obsolet, refer to specs to use the correct one depending on case:

Activity indicators are specified by use case, please refer to this link:
https://wiki.mozilla.org/Gaia/Design/BuildingBlocks#Progress_.26_Activity_Indicators

Building blocks for consolidating this are implemented and approval pending:
https://bugzilla.mozilla.org/show_bug.cgi?id=822671

Comment 1

[Eric Pang [:epang] UX]

Hi Victoria, looked at the building blocks, should the light progress and activity bard be used?  If so, Pavel can you update to use the correct building block? Thanks!

Comment 2

[Victoria Gerchinhoren [:vicky]]

Eric:
Yes, that's right.

Comment 3

[Eric Pang [:epang] UX]

Attachment: 11_OWD_Progress_Activity_Components.jpg

Hey Pavel, can you update activity indicator to the light in the attached image?  It should be part of the building blocks.

Comment 4

[Pavel Ivanov [:ivanovpavel][:pivanov] UX]

Hey Eric,
can you suggest me which version we need to use here ... the light or the dark one, because it's not clear here for this case:

"Note that the activiy bas is background sensitive and there is an option for light screens and another for the dark ones."

Comment 5

[Eric Pang [:epang] UX]

(In reply to Pavel Ivanov [:ivanovpavel] from comment #4)
> Hey Eric,
> can you suggest me which version we need to use here ... the light or the
> dark one, because it's not clear here for this case:
> 
> "Note that the activiy bas is background sensitive and there is an option
> for light screens and another for the dark ones."

I believe we should use the light one.  Let's see what it looks like, if needed we can switch it.  Thanks!

Comment 6

[Pavel Ivanov [:ivanovpavel][:pivanov] UX]

Hey Gareth,
I have a question ... we have the progress bar on this line https://github.com/mozilla-b2g/gaia/blob/master/apps/calendar/index.html#L125 ... but I'm not sure that we use this now ... I can not find any JS logic who show/hide this progress bar ... can you confirm? Because if we remove the logic for some reason I can put the right progress bar and also we should open a new bug for the logic one, right?

Comment 7

[Gareth Aye [:gaye] (back from PTO)]

Hey Pavel - I'll take a look later today! Sorry for the slow turnaround.

Comment 8

[Eric Pang [:epang] UX]

This will be fixed with a new bug we opened for the 1.3 visual refresh.