Closed Bug 1100764 Opened 7 years ago Closed 7 years ago
Contacts move up and down when hovering over them
See attached screencast. STR: Add two contacts Move mouse between the two back and forth ER: The individual contacts do not move. AR: The contacts jump around like they have ants in their paints
This is fine on Mac, so I suspect this is windows-only.
OS: All → Windows 8
Assignee: nobody → jaws
Status: NEW → ASSIGNED
Iteration: --- → 36.3
Points: --- → 3
A contact has a height of 51px, with 5px padding-top and 5px padding-bottom. This leaves the contact with 41px for the content box. When a contact is hovered, we show icons next to the contact's details. Previously, this box had a height of 43px, which included 12px padding-top and 12px padding-bottom. Since the 43px height of the icons box was larger than the 41px for contact's content-box, the contact grew in size by 2px when hovered. This patch changes the icon to have 10px of padding on all sides (it previously used 10px left and right), and thus the height of the icon box will now be 39px, smaller than the 41px content-box of the contact and doesn't cause contacts to grow in height when hovered. I also changed some `background` properties in the CSS to use `background-color` since they didn't need to be using the shorthand.
Attachment #8525645 - Flags: review?(mdeboer)
Comment on attachment 8525645 [details] [diff] [review] Patch Review of attachment 8525645 [details] [diff] [review]: ----------------------------------------------------------------- LGTM. Ship it!
Attachment #8525645 - Flags: review?(mdeboer) → review+
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
Comment on attachment 8525645 [details] [diff] [review] Patch Approval Request Comment [Risks and why]: Loop rooms code needed for 35 pref-on CSS-only change that cleans up a UI issue. [String/UUID change made/needed]: none
Attachment #8525645 - Flags: approval-mozilla-aurora?
Attachment #8525645 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.