Skip to content

refactor: move bookmark glyph into shared icon - #265

Merged
johannesjo merged 1 commit into
johannesjo:mainfrom
zerone0x:zerone0x/shared-icons-208
Sep 3, 2026
Merged

refactor: move bookmark glyph into shared icon#265
johannesjo merged 1 commit into
johannesjo:mainfrom
zerone0x:zerone0x/shared-icons-208

Conversation

@zerone0x

Copy link
Copy Markdown
Contributor

Summary

  • add BookmarkIcon to the shared icon module
  • replace the duplicated inline bookmark SVGs in the terminal bookmark gutter
  • keeps the two existing rendered sizes (15px and 14px) while removing the local pasted path

Refs #208

Tests

  • npm ci --ignore-scripts (plain npm ci failed because node-pty native postinstall could not rebuild in this container)
  • npm run typecheck
  • git diff --check

@johannesjo

Copy link
Copy Markdown
Owner

Thank you very much! <3

@johannesjo
johannesjo merged commit 4ba82cb into johannesjo:main Sep 3, 2026
2 checks passed
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.

2 participants