Skip to content

feat(icon): DLT-3136 convert docs inline svgs to components#1123

Merged
Josh Everhart (jeverhart-dialpad) merged 2 commits intostagingfrom
DLT-3136-convert-svgs-to-components
Mar 13, 2026
Merged

feat(icon): DLT-3136 convert docs inline svgs to components#1123
Josh Everhart (jeverhart-dialpad) merged 2 commits intostagingfrom
DLT-3136-convert-svgs-to-components

Conversation

@jeverhart-dialpad
Copy link
Contributor

feat(icon): DLT-3136 convert docs inline svgs to components

Obligatory GIF (super important!)

Obligatory GIF

🛠️ Type Of Change

These types will increment the version number on release:

  • Fix
  • Feature
  • Performance Improvement
  • Refactor

These types will not increment the version number, but will still deploy to documentation site on release:

  • Documentation
  • Build system
  • CI
  • Style (code style changes, not css changes)
  • Test
  • Other (chore)

📖 Jira Ticket

https://dialpad.atlassian.net/browse/DLT-3136

📖 Description

Move inline SVG icons that appear in the documentation site into the dialtone icons package so they can more easily be re-used. These include Storybook (color & mono), Github, Codepen, and Figma.

Currently used in the top nav bar, page headers for components, and a popover displayed in the brand page.

💡 Context

The Dialtone docs site uses a series of brand icons in the top navigation and also in inner page headers. Those icons are currently inline SVGs.

📝 Checklist

For all PRs:

  • I have ensured no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).
  • I have reviewed my changes.
  • I have added all relevant documentation.
  • I have considered the performance impact of my change.

🔮 Next Steps

Update usage on the UI kits landing page in the next branch (after staging is merged into next in the future).

📷 Screenshots / GIFs

Screenshot 2026-03-13 at 8 24 52 AM Screenshot 2026-03-13 at 8 25 09 AM Screenshot 2026-03-13 at 8 25 24 AM

🔗 Sources

@github-actions
Copy link
Contributor

Please add either the visual-test-ready or no-visual-test label to this PR depending on whether you want to run visual tests or not.
It is recommended to run visual tests if your PR changes any UI. ‼️

@github-actions
Copy link
Contributor

✔️ Deploy previews ready!
😎 Dialtone documentation preview: https://dialtone.dialpad.com/deploy-previews/pr-1123/
😎 Dialtone-vue preview: https://dialtone.dialpad.com/vue/deploy-previews/pr-1123/

@jeverhart-dialpad Josh Everhart (jeverhart-dialpad) added the visual-test-ready Add this tag when the PR is ready for visual test, to trigger GHA visual tests label Mar 13, 2026
@jeverhart-dialpad
Copy link
Contributor Author

🗒️ Percy flagged changes on the storybook icon variants page (https://dialtone.dialpad.com/vue/deploy-previews/pr-1123/?path=/story/components-icon--variants)

Screenshot 2026-03-13 at 9 16 01 AM

These are expected, as the codepen icon would now show in that flagged area, causing the shift.

Copy link
Contributor

Choose a reason for hiding this comment

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

Nice call. Merge this right off your plate.

@jeverhart-dialpad Josh Everhart (jeverhart-dialpad) merged commit 8bae326 into staging Mar 13, 2026
22 checks passed
@jeverhart-dialpad Josh Everhart (jeverhart-dialpad) deleted the DLT-3136-convert-svgs-to-components branch March 13, 2026 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

visual-test-ready Add this tag when the PR is ready for visual test, to trigger GHA visual tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants