Problem

Block visibility summaries used to work as regular vertical tabs summaries providing a short overview of the block visibility setup. The fields were renamed, but visibility JS was not updated in #1535868: Convert all blocks into plugins.

Proposal

Fix the JS by updating it to the right selectors. Also remove user based visibility JS since the feature was removed. #1871854: Decide whether to restore per-user block visibility or provide it in contrib).

SummaryBefore.png

SummaryAfter.png

CommentFileSizeAuthor
block-visibility-js.patch1.45 KBgábor hojtsy
SummaryAfter.png52.66 KBgábor hojtsy
SummaryBefore.png47.13 KBgábor hojtsy

Comments

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented

Although not depicted on the screenshot, I cross-checked with language visibility as well and that works too.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Title: Block visibility summaries are broken » Regression: Block visibility summaries are broken

Titling to call regression into focus.

ParisLiakos’s picture

ParisLiakos commented
Status: Needs review » Reviewed & tested by the community

nice catch! i would have never notice it

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to 8.x. Thanks.

xjm’s picture

xjm
she/her
Primary language English
commented
Issue tags: +Block plugins

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

Anonymous’s picture

(not verified) commented
Issue summary: View changes

Updated issue summary.