Skip to content

[editor/diff] Update toggle icon size in search panel#826

Merged
mdo merged 1 commit into
beta-1.3from
editor/fix-search-ui
Jun 17, 2026
Merged

[editor/diff] Update toggle icon size in search panel#826
mdo merged 1 commit into
beta-1.3from
editor/fix-search-ui

Conversation

@ije

@ije ije commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator
  • Reduce the icon size from 16 -> 14
  • Reduce the width from 24 -> 20
Screenshot 2026-06-17 at 15 19 54

Screenshot 2026-06-17 at 15 22 53

@vercel

vercel Bot commented Jun 17, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
pierre-docs-diffs Ready Ready Preview Jun 17, 2026 7:24am
pierre-docs-diffshub Ready Ready Preview Jun 17, 2026 7:24am
pierre-docs-trees Ready Ready Preview Jun 17, 2026 7:24am
pierrejs-diff-demo Ready Ready Preview Jun 17, 2026 7:24am

Request Review

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

Updates the diffs editor search panel toggle controls to better match the intended visual density by shrinking the toggle glyphs and their button width.

Changes:

  • Render the search option toggle icons (case/whole-word/regex) at 14px instead of the default 16px.
  • Reduce the width of icon buttons inside the search input box from 24px to 20px via CSS.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
packages/diffs/src/editor/searchPanel.ts Passes an explicit 14px size for the search option toggle SVGs.
packages/diffs/src/editor/editor.css Narrows icon-button width within [data-input-box] to 20px to reduce toggle footprint.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mdo
mdo merged commit 9579e97 into beta-1.3 Jun 17, 2026
9 checks passed
@mdo
mdo deleted the editor/fix-search-ui branch June 17, 2026 15:00
tjni pushed a commit to tjni/pierre that referenced this pull request Jun 25, 2026
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