The Bootstrap theme uses (button) elements instead of (input type=button) and the admin js for adding displays is too specific to find it.

CommentFileSizeAuthor
#6 views-ui-button-js-2402655-6.patch796 bytesaDarkling
#1 views-ui-button-js-2402655-1.patch796 bytesfrankcarey
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

frankcarey’s picture

frankcarey CreditAttribution: frankcarey commented
Status: Active » Needs review
FileSize
views-ui-button-js-2402655-1.patch796 bytes

This simply removes the element type from the selector and then all is well.

frankcarey’s picture

frankcarey CreditAttribution: frankcarey commented
Related issues: +#1834688: Views UI problems
ckng’s picture

ckng
Primary language English
CreditAttribution: ckng commented
Status: Needs review » Reviewed & tested by the community

Patch #1 working as expected.

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented

Yes please, RTBC++

colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
CreditAttribution: colan at Colan Schwartz Consulting for Chapter Three commented

We've recently switched our testing from the old qa.drupal.org to DrupalCI. Because of a bug in the new system, #2623840: Views (D7) patches not being tested, older patches must be re-uploaded. On re-uploading the patch, please set the status to "Needs Review" so that the test bot will add it to its queue.

If all tests pass, change the Status back to "Reviewed & tested by the community". We'll most likely commit the patch immediately without having to go through another round of peer review.

We apologize for the trouble, and appreciate your patience.

aDarkling’s picture

aDarkling CreditAttribution: aDarkling as a volunteer commented
FileSize
views-ui-button-js-2402655-6.patch796 bytes

This is the same #1 patch by frankcarey.
Just re-uploading it as directed to get this issue closed.

aDarkling’s picture

aDarkling CreditAttribution: aDarkling as a volunteer commented
Status: Reviewed & tested by the community » Needs review
colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
CreditAttribution: colan at Colan Schwartz Consulting for Chapter Three commented

Is this also a problem in D8? If so, we'll need to create a separate issue for it.

aDarkling’s picture

aDarkling CreditAttribution: aDarkling as a volunteer commented
Status: Needs review » Reviewed & tested by the community

Don't know. I'm still wrapped up in D7 projects.

kopeboy’s picture

kopeboy CreditAttribution: kopeboy as a volunteer commented

please include in next release, very simple and works.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Reviewed & tested by the community » Fixed
Parent issue: » #3054023: Plan for Views 7.x-3.24 release

Committed. Thanks all.

Status: Fixed » Closed (fixed)

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