This is a follow-on to #1093550: Replace "Edit" text with Contextual Links - it seems fairly unfriendly to remove the regular "Configure block" link from the set of contextual links, since users may in fact want to configure the block and not just edit the content. It would be preferable to follow the example of, say, Views, and display your "Edit block" link in addition rather than instead.

CommentFileSizeAuthor
#9 boxes-configure-link-2162465-9.patch419 bytesxaqrox
#7 boxes-configure-link-2162465-7.patch419 bytesxaqrox
#3 boxes-configure-link-2162465-3.patch392 bytesbrad.bulger
#1 boxes-configure-link-2162465.patch391 bytesbrad.bulger
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

brad.bulger’s picture

brad.bulger CreditAttribution: brad.bulger commented
FileSize
boxes-configure-link-2162465.patch391 bytes

It's just a one-line removal.

antongp’s picture

antongp CreditAttribution: antongp commented

+1 . Core's "Configure block" link should be there

brad.bulger’s picture

brad.bulger CreditAttribution: brad.bulger commented
FileSize
boxes-configure-link-2162465-3.patch392 bytes

reroll against current 1.x branch

Alan D.’s picture

Alan D. CreditAttribution: Alan D. commented
Status: Active » Reviewed & tested by the community

Strange, you create a block replacement and then physically remove the mechanism to configure it... I assume that there was a valid reason for this? Let us know and we can retest that use-case.

As is, it seems that this is RTBTC :)

mqanneh’s picture

mqanneh
he/him
Primary language English
Location Amman
CreditAttribution: mqanneh as a volunteer and commented

+1

joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented

RTBC++ , this patch works great, thanks!

xaqrox’s picture

xaqrox
he/him
Primary language English
Location Washington, D.C.
CreditAttribution: xaqrox commented
FileSize
boxes-configure-link-2162465-7.patch419 bytes

The ul.boxes.box-controls is picking up styles from my theme that are causing problems. Since it's not being used, thought it made sense to hide it.

xaqrox’s picture

xaqrox
he/him
Primary language English
Location Washington, D.C.
CreditAttribution: xaqrox commented
Status: Reviewed & tested by the community » Needs review
xaqrox’s picture

xaqrox
he/him
Primary language English
Location Washington, D.C.
CreditAttribution: xaqrox commented
FileSize
boxes-configure-link-2162465-9.patch419 bytes

Sorry, bad patch before.