Skip to content

fix(overview): keep the app grid clear of the dock - #38

Open
perlowja wants to merge 1 commit into
singularityos-lab:mainfrom
perlowja:pr/overview-dock-clearance-20261014
Open

perlowja wants to merge 1 commit into
singularityos-lab:mainfrom
perlowja:pr/overview-dock-clearance-20261014

Conversation

@perlowja

Copy link
Copy Markdown
Contributor

The overview is a full-output layer surface with exclusive zone -1, and the dock stays above it while the overview is open, also with autohide and intellihide. The grid used the full output height, so the dock covered the bottom row of app icons. A side dock was only accounted for when autohide was off.

This adds Dock.shown_extent() (the exclusive zone the dock claims when always visible, plus its layer margin) and insets the overview content by it on the dock's edge. The grid's viewport hint and column count use the area that is left, and search results get the same inset.

Before: with the default bottom dock the last row of icons sat under the dock at every scale we tried (1920x1080 at 1x, 3840x2160 at 1.75x and 2x, 3072x1920 at 2x).
After: the last row ends above the dock and the grid scrolls as before, on the same outputs, on amd64 and arm64.

Not changed: overviews on secondary monitors (they have no dock).

Build and tests on Debian forky: ninja builds, meson test 82 of 84 pass, the same as main (parental-helper and crash-handler fail identically on main in a container).

AI assistance: disclosed

The overview is a full-output layer surface with exclusive zone -1, and
the dock stays above it on the overlay layer, shown while the overview
is open even in autohide and intellihide modes. The grid viewport ran
under the dock and the dock covered the bottom row of icons; a side
dock was only accounted for when autohide was off.

Add Dock.shown_extent(), the distance the shown dock reaches in from its
edge (the exclusive zone it claims when always visible plus its layer
margin), and inset the overview content stack by it on the dock edge.
The grid viewport height hint and the column count use the remaining
area. Search results get the same inset.

Assisted-by: Claude Code:claude-opus-5-5
AI-Scope: Located the overview/dock layering cause and wrote the dock inset change; reviewed and tested on hardware by the author.
@perlowja

Copy link
Copy Markdown
Contributor Author

@mirkobrombin for review when you have a moment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant