Closed
Bug 1081062
Opened 11 years ago
Closed 10 years ago
Contact call/menu button click areas are too small
Categories
(Hello (Loop) :: Client, defect, P4)
Hello (Loop)
Client
Tracking
(Not tracked)
RESOLVED
WORKSFORME
backlog | backlog+ |
People
(Reporter: standard8, Unassigned)
References
Details
(Whiteboard: [UX bug])
Attachments
(1 file)
1.48 KB,
patch
|
Details | Diff | Splinter Review |
The call contact button / menu drop-down have very small areas where you can click on them compared to the actual size of the clickable area of the camera icon and the triangle.
The hover changes as soon as you enter the green area, but you're not able to select anything until you get very close the the icons themselves.
Reporter | ||
Updated•11 years ago
|
OS: Mac OS X → All
Hardware: x86 → All
Updated•11 years ago
|
backlog: --- → Fx36?
![]() |
||
Updated•11 years ago
|
backlog: Fx36? → Fx37+
Priority: -- → P4
![]() |
||
Comment 1•11 years ago
|
||
#3 priority from Contacts review, try for 36 - no strings
backlog: Fx37+ → Fx36+
Priority: P4 → P3
Whiteboard: [UX]
![]() |
||
Comment 2•11 years ago
|
||
#3 priority from Contacts review, try for 36 - no strings
![]() |
||
Updated•11 years ago
|
Priority: P3 → P4
Comment 3•11 years ago
|
||
Comment 4•11 years ago
|
||
Resetting the priority since P4 now indicates a minor bug. (P4 used to indicate a UX issue.)
If we can, I'd love to uplift this to Fx36, but only if someone take it in time.
backlog: Fx36+ → Fx37+
Priority: P4 → P2
Comment 5•11 years ago
|
||
(In reply to Andrei Oprea [:andreio] from comment #3)
> Created attachment 8531793 [details] [diff] [review]
> Loop contacts button are too small
Hi Andrei, what are the details of this patch you submitted?
Comment 6•11 years ago
|
||
Fix: http://cl.ly/image/0K1J24060g2P
Move the arrow over slightly and make the entire right-portion of the button a clickable area.
![]() |
||
Comment 7•11 years ago
|
||
does this just need a reviewer to approve Andrei's patch? If yes - good ideas on who to assign so we can land?
Flags: needinfo?(mreavy)
Flags: needinfo?(gavin.sharp)
Comment 8•11 years ago
|
||
Mark -- Can you take a quick peak at the patch on this bug? I believe Andrei is part-way there, but I'd appreciate your assessment of where this bug stands.
Flags: needinfo?(standard8)
Flags: needinfo?(mreavy)
Flags: needinfo?(gavin.sharp)
![]() |
||
Comment 9•11 years ago
|
||
moving to 38 until up for review. the P4 is just based on new priorities - which is often where UX falls
backlog: Fx37+ → Fx38+
Priority: P2 → P4
Reporter | ||
Comment 10•11 years ago
|
||
I took a look at the patch, but either the bitrot & recent changes have broken it, or the layout of the icons is still wrong.
Flags: needinfo?(standard8)
![]() |
||
Updated•11 years ago
|
backlog: Fx38+ → backlog+
Rank: 47
Flags: firefox-backlog+
Whiteboard: [UX] → [UX bug]
Reporter | ||
Comment 11•10 years ago
|
||
With the UX refresh, I think this is good enough that we can call it works for me.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•