Skip to content

Map Layout Zoom Control Spacing Fix - #1299

Open
jakeymac wants to merge 2 commits into
mainfrom
map-zoom-button-fix
Open

Map Layout Zoom Control Spacing Fix#1299
jakeymac wants to merge 2 commits into
mainfrom
map-zoom-button-fix

Conversation

@jakeymac

@jakeymac jakeymac commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Description

This merge request addresses two spacing issues in MapLayout:

  • A gap between the "fit to extent" and zoom out buttons in the map zoom controls.
  • Extra empty space at the bottom of the page when the MapLayout has a subtitle or a multi-line title

See images below:

Changes Made to Code

  • Adjust styling to remove empty space between the 'fit to extent' and zoom out buttons.
  • Adjust styling to remove empty space along the bottom of the page.

Before:
image

After:
image

Before:
Screenshot 2026-08-19 100211

After:
Screenshot 2026-08-19 100120

Quality Checks

  • At least one new test has been written for new code
  • New code has 100% test coverage
  • Code has been formatted with Black
  • Code has been linted with flake8
  • Docstrings for new methods have been added
  • The documentation has been updated appropriately

@coveralls

coveralls commented Aug 17, 2026

Copy link
Copy Markdown

Coverage Status

coverage: 100.0%. remained the same — map-zoom-button-fix into main

@jakeymac
jakeymac marked this pull request as ready for review August 17, 2026 23:18
@jakeymac jakeymac self-assigned this Aug 17, 2026
@jakeymac
jakeymac requested review from swainn and a lite review from Copilot August 17, 2026 23:18

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adjusts the OpenLayers zoom/extent control styling in MapLayout to eliminate the unwanted gap between the “fit to extent” control and the zoom-out button.

Changes:

  • Update .ol-zoom-extent positioning from a fixed px value to an em-based offset (top: 2.75em) for consistent spacing.
  • Align the .ol-zoom-extent button height with the standard control button height (2.25em) to remove the leftover gap.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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.

3 participants