Closed Bug 1044996 Opened 10 years ago Closed 9 years ago

[Dialer][Call screen] New hit state for call screen options (mute, place new call, keypad...)

Categories

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

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: paco, Unassigned)

Details

Attachments

(3 files, 2 obsolete files)

      No description provided.
Assignee: nobody → pacorampas
OS: Mac OS X → All
Hardware: x86 → All
Target Milestone: --- → 2.1 S1 (1aug)
Attached image hit-state.png (obsolete) —
Attachment #8464493 - Flags: feedback?(b.pmm)
Attachment #8464493 - Flags: feedback?(b.pmm) → feedback+
Attached file Demo (youtube) (obsolete) —
Attachment #8464530 - Flags: feedback?(b.pmm)
Hey Carol, what do you think about our proposal for the new hit states?
Flags: needinfo?(chuang)
Attachment #8464530 - Flags: feedback?(b.pmm) → feedback+
Attached image hit-state
Attachment #8464493 - Attachment is obsolete: true
Attachment #8464553 - Flags: feedback?
Attachment #8464553 - Flags: feedback? → feedback?(b.pmm)
Attachment #8464553 - Flags: feedback?(b.pmm) → feedback+
Hi Pau,
I saw your demo on the new hit state. The interaction is good.

But the "Hit state" should be consistent across the whole system. Przemek is already working on our building blocks including the basic hit state. I'll ni Przemek to comment on this.
Thank you!
Flags: needinfo?(padamczyk)
Flags: needinfo?(pabratowski)
Flags: needinfo?(chuang)
Attached file demo (youtube)
Attachment #8464530 - Attachment is obsolete: true
Attachment #8464601 - Flags: feedback?(b.pmm)
Attachment #8464601 - Flags: feedback?(b.pmm) → feedback+
Looks good, I'll let Przemek have the final say since he owns these components.
Flags: needinfo?(padamczyk)
Comment on attachment 8464601 [details]
demo (youtube)

It's cool but this would be the only app doing this. We're coming up with an action icon hit state behavior for 2.2 that should be used across the UI but that won't be until 2.2. Are these changes for 2.1 or 2.2? Casey can you send a link to the newest 2.2 action icon hit state behavior?
Attachment #8464601 - Flags: feedback+ → feedback-
Flags: needinfo?(pabratowski) → needinfo?(kyee)
Whiteboard: [planned-sprint c=]
Target Milestone: 2.1 S1 (1aug) → 2.1 S2 (15aug)
Whiteboard: [planned-sprint c=] → [planned-sprint c=][in-sprint=v2.1-S1]
Whiteboard: [planned-sprint c=][in-sprint=v2.1-S1] → [planned-sprint c=][in-sprint=v2.1-S2]
Target Milestone: 2.1 S2 (15aug) → ---
Whiteboard: [planned-sprint c=][in-sprint=v2.1-S2]
It appears that Casey has left. Przemek, has there been any progress on a decision here? We'd be happy to discuss this further.
Flags: needinfo?(kyee) → needinfo?(pabratowski)
You can currently see the icon hit state behavior in the 2.1 application headers. Wilson should be able to give you the specific hit state code and timing from the header web component.
Flags: needinfo?(pabratowski) → needinfo?(wilsonpage)
(In reply to Przemek Abratowski [:przemek] UX from comment #11)
> You can currently see the icon hit state behavior in the 2.1 application
> headers. Wilson should be able to give you the specific hit state code and
> timing from the header web component.

Ok, so are you saying that we should use the same hit state behavior as the header component has for this case? If so, I don't think we have anything to do here.
(In reply to Przemek Abratowski [:przemek] UX from comment #11)
> You can currently see the icon hit state behavior in the 2.1 application
> headers. Wilson should be able to give you the specific hit state code and
> timing from the header web component.

We're using JS [1] to apply and remove our own `.active` classes as CSS wouldn't do what we wanted. This code is currently bundled in gaia-header, but I'm likely going to pull it out into its own module as other components will likely need it. For consistency across the OS, we really need this logic in one place.

[1] https://github.com/gaia-components/gaia-header/blob/master/gaia-header.js#L505-L533
Flags: needinfo?(wilsonpage)
(In reply to Wilson Page [:wilsonpage] from comment #13)
> We're using JS [1] to apply and remove our own `.active` classes as CSS
> wouldn't do what we wanted. This code is currently bundled in gaia-header,
> but I'm likely going to pull it out into its own module as other components
> will likely need it. For consistency across the OS, we really need this
> logic in one place.

Thanks. Is there a bug tracking this?
Assignee: pacorampas → nobody
We're doing this in Web Components. The <gaia-button> component can be overridden however we need.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: