Skip to content

chore: remove vertical spacer in map legends - #275

Merged
ralvarez-dg merged 2 commits into
mainfrom
task/remove-vertical-spacer-in-map-legend
Jul 28, 2026
Merged

chore: remove vertical spacer in map legends#275
ralvarez-dg merged 2 commits into
mainfrom
task/remove-vertical-spacer-in-map-legend

Conversation

@timothygachengo

Copy link
Copy Markdown
Contributor

Description

Type of change

  • Bug fix (fix:)
  • New feature (feat:)
  • Breaking change (BREAKING CHANGE:)
  • Refactor / chore (refactor: / chore:)
  • Documentation update (docs:)

Affected package(s)

  • @devgateway/dvz-ui-react
  • @devgateway/wp-react-lib
  • example only (no changeset needed)

Checklist

  • PR title follows Conventional Commits format
  • A changeset has been added (pnpm changeset) for any change to a published package
  • pnpm build passes locally
  • pnpm --filter @devgateway/* typecheck passes
  • No hardcoded credentials, internal URLs, client names, or PII introduced
  • Any new dependency has an Apache-2.0-compatible license

Copilot AI review requested due to automatic review settings July 28, 2026 17:07

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Removes the visual “vertical spacer” separators (|) from the embeddable map legend UI and includes a changeset to publish the adjustment in @devgateway/dvz-ui-react.

Changes:

  • Remove the conditional title spacer element from the legend header.
  • Remove the per-item spacer element from the symbols legend list.
  • Add a patch changeset for @devgateway/dvz-ui-react.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
packages/dvz-ui/src/embeddable/map/legend.jsx Removes the vertical spacer elements from legend title and symbol items.
.changeset/dry-clubs-stand.md Adds a patch changeset describing the legend spacer removal.
Comments suppressed due to low confidence (1)

.changeset/dry-clubs-stand.md:6

  • The changeset file ends with a whitespace-only line. Remove the trailing whitespace/extra blank line to keep changeset files clean and avoid whitespace-only diffs.
Remove vertical spacer in map legend

Comment thread packages/dvz-ui/src/embeddable/map/legend.jsx
@ralvarez-dg
ralvarez-dg merged commit 529fb32 into main Jul 28, 2026
2 checks passed
@timothygachengo
timothygachengo deleted the task/remove-vertical-spacer-in-map-legend branch July 28, 2026 17:10
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