Skip to content

refactor: icons.#34

Merged
knightedcodemonkey merged 2 commits intomainfrom
bananas
Mar 27, 2026
Merged

refactor: icons.#34
knightedcodemonkey merged 2 commits intomainfrom
bananas

Conversation

@knightedcodemonkey
Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings March 27, 2026 23:20
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Refactors UI icons across the app to use consistent inline SVGs (instead of text glyphs / mixed icon styles) and updates related styling so icons inherit currentColor cleanly.

Changes:

  • Replace token info / add icons with SVGs (and update token placeholder text to “GitHub PAT”).
  • Update editor tools toggle icons to a new SVG and adjust fill/stroke styling.
  • Add an icon to the “GitHub” mobile toggle and tune layout spacing/alignment.

Reviewed changes

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

Show a summary per file
File Description
src/styles/panels-editor.css Switch editor tools toggle icons to filled SVG styling.
src/styles/layout-shell.css Add SVG sizing + spacing for the mobile “GitHub” toggle button.
src/styles/ai-controls.css Restyle token info/add buttons to match filled SVG icon approach.
src/modules/github-byot-controls.js Swap token info button from text (i/?) to SVG icons based on token state.
src/index.html Replace inline text icons with SVGs; update token input placeholder wording.

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

@knightedcodemonkey knightedcodemonkey merged commit 919d0df into main Mar 27, 2026
1 check passed
@knightedcodemonkey knightedcodemonkey deleted the bananas branch March 27, 2026 23:41
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