In the current state, the hover dropdown plugin provided by bedrock_views doesn't work if the user selects to not activate the dropdown on hover, i.e. click insteade. Also, the markup in the template is a bit off, and the configuration is missing a third state (not in the trigger and not in the dropdown). We want to shore up the markup to work with Foundation 5, and allow it to work for both click and hover triggers.

CommentFileSizeAuthor
#3 bedrock_views--broken-click-dropdown--2285365-3.patch10.47 KBmpgeek
#1 bedrock_views--broken-click-dropdown--2285365-1.patch10.48 KBmpgeek

Comments

mpgeek’s picture

mpgeek commented
StatusFileSize
new bedrock_views--broken-click-dropdown--2285365-1.patch10.48 KB

Here is a patch for that. Note that at this time the latest dev commit is 1064386, which is also tag alpha5. The patch will not apply to alpha5 but it will apply to latest dev.

mpgeek’s picture

mpgeek commented
Status: Active » Needs review
mpgeek’s picture

mpgeek commented
StatusFileSize
new bedrock_views--broken-click-dropdown--2285365-3.patch10.47 KB

Patch at #1 had extraneous classes. This patch uses minimal classes on the dropdown content wrapper.

  • Commit 9f4da16 on 7.x-1.x authored by mpgeek, committed by anniegreens: 
    Issue #2285365 by mpgeek: Fixed Dropdown button views row style doesn't...
jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented
Status: Needs review » Fixed

Marking fixed.

Status: Fixed » Closed (fixed)

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