If you are using admin module and the context_ui module, or if you have positioned the context-editor_ui block into one of your regions, empty blocks show up saying "This block appears empty when displayed on this page.". This is great because it means that you can manage the position of those blocks, and it means that empty block's position doesn't get lost when you resave a content. However, its very distracting having these show.

Attached patch adds/removes the context-block-hidden class when entering/exiting the context inline editor. This means that the empty blocks only show up when you are actually doing inline context editing.

I think the problem is similar to that described in #872564: Empty blocks are shown though not exactly the same, and that issue if for D6.

CommentFileSizeAuthor
#2 1401964.2-context-hide-empty-block.patch2.72 KBmrfelton
#1 1401964.1-context-hide-empty-block.patch1.61 KBmrfelton
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mrfelton’s picture

mrfelton CreditAttribution: mrfelton commented
Status: Active » Needs review
FileSize
1401964.1-context-hide-empty-block.patch1.61 KB
mrfelton’s picture

mrfelton CreditAttribution: mrfelton commented
FileSize
1401964.2-context-hide-empty-block.patch2.72 KB

This patch works better, ensure that the entire block is hidden/shown, and not just the block content.

mrfelton’s picture

mrfelton CreditAttribution: mrfelton commented

note - clear your cache after applying to ensure the new preprocess function gets picked up.

tekante’s picture

tekante CreditAttribution: tekante commented
Status: Needs review » Fixed

Committed, thanks for the bug report and patch

Status: Fixed » Closed (fixed)

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