Problem/Motivation

Since RefreshLess forces all regions to render, this can cause layout issues on some themes. Having some way to identify regions that don't contain anything (other than the RefreshLess placeholder) would help a lot.

Steps to reproduce

Install RefreshLess, use a theme that does layout stuff that can cause empty but still rendering regions to take up space. Some examples include flexbox or CSS grid, or using the + combinator, e.g. .region + .region for margins.

Proposed resolution

Give regions that contain only the RefreshLess placeholder the hidden attribute, which will hide the region in modern browsers and also make the region identifiable via CSS or JavaScript via the [data-refreshless-region][hidden] selector.

Remaining tasks

Add the hidden attribute to empty regions.

User interface changes

None?

API changes

None?

Data model changes

None.

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

Ambient.Impact created an issue. See original summary.

ambient.impact’s picture

Assigned: ambient.impact » Unassigned
Status: Active » Needs review
rajeshreeputra’s picture

Ambient.Impact I have given you access to the project to maintain it, go ahead!

ambient.impact’s picture

Issue summary: View changes
ambient.impact’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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