The hover state for dates in the date editing popup has different dimensions than the passive state. This creates a disturbing rippling effect as you mouseover dates in the popup.

This is what the popup calendar should look like:

Screenshot 1

And here is what it looks like when you hover with Panopoly:

Screenshot 2
Screenshot 3

CommentFileSizeAuthor
#6 panopoly_core-date-popup-hover-2541570-6.patch462 bytesdsnopek
#2 date-popup-hover-issue-2541570-2.patch458 bytesgdaw
Screen Shot 2015-07-28 at 4.02.27 PM.png52.27 KBhumansky
Screen Shot 2015-07-28 at 4.02.21 PM.png48.68 KBhumansky
Screen Shot 2015-07-28 at 4.02.10 PM.png52.29 KBhumansky
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
CreditAttribution: dsnopek commented
Component: User interface » Core

Hm. That sounds like it's an issue with the jQuery UI theme that is bundled in panopoly_core. It was created with Theme Roller, so one would hope it wouldn't have weird sizing issues, but we also made a couple manual modifications which are probably to blame.

gdaw’s picture

gdaw CreditAttribution: gdaw as a volunteer commented
FileSize
date-popup-hover-issue-2541570-2.patch458 bytes

Confirmed that the jittery date boxes was caused by some CSS borders added on hover but not on default. Fixed by adding the same borders to default date boxes. See patch file.

gdaw’s picture

gdaw CreditAttribution: gdaw as a volunteer commented
Status: Active » Needs review
bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
CreditAttribution: bkosborne at Princeton University commented
Status: Needs review » Reviewed & tested by the community

We've been using this patch for months without issue.

byronveale’s picture

byronveale CreditAttribution: byronveale at Princeton University commented

*bump*

(not that you aren't busy enough…!)

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
CreditAttribution: dsnopek commented
Status: Reviewed & tested by the community » Needs review
FileSize
panopoly_core-date-popup-hover-2541570-6.patch462 bytes

Thanks for digging up this issue! I'd like to propose a slightly different fix, which solves the spacing issue, but causes less drastic changes to the current visual appearance.

  • dsnopek committed 36dd382 on 7.x-1.x 
    Update Panopoly Core for Issue #2541570 by dsnopek, gdaw, humansky,...
dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
CreditAttribution: dsnopek commented
Status: Needs review » Fixed

I went ahead and commited this one just to keep things moving forward, but if anyone objects to my alternate patch in #6 please feel free to reopen and comment here! If you do so in the next day, it'll probably be in time to revert before the next release, if necessary

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.