Skip to content

Conversation

@spike-rabbit
Copy link
Member

Adds the icon-viewer to element.
Does some technical overhaul compared to the old internal version.


@spike-rabbit spike-rabbit force-pushed the docs/icon-viewer branch 5 times, most recently from 2f0c64c to 84235f6 Compare January 9, 2026 13:19
@github-actions
Copy link

github-actions bot commented Jan 9, 2026

@spike-rabbit spike-rabbit force-pushed the docs/icon-viewer branch 5 times, most recently from 4a71003 to 8fbe152 Compare January 23, 2026 08:22
@spike-rabbit spike-rabbit marked this pull request as ready for review January 23, 2026 08:36
@spike-rabbit spike-rabbit requested review from a team as code owners January 23, 2026 08:36
Copy link
Member Author

@spike-rabbit spike-rabbit left a comment

Choose a reason for hiding this comment

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

Updated the icon viewer docs and standalone page per your feedback:

Let me know if anything else should change.

@spike-rabbit
Copy link
Member Author

Updated the icon viewer docs and standalone page per your feedback:

Let me know if anything else should change.

@spliffone
Copy link
Member

@Killusions Do you know why do we need the brand-banner.html I can't find it referenced nor can I see it in the network tab?

@Killusions
Copy link
Member

@Killusions Do you know why do we need the brand-banner.html I can't find it referenced nor can I see it in the network tab?

We don't need it, was just for simpl/simpl

@spliffone spliffone force-pushed the docs/icon-viewer branch 3 times, most recently from ebee6a0 to 817f64e Compare January 25, 2026 10:47
@spike-rabbit spike-rabbit force-pushed the docs/icon-viewer branch 3 times, most recently from 92e05d9 to dac8d41 Compare January 28, 2026 16:09
@timowolf timowolf added this to the 49.0.0 milestone Jan 30, 2026
@timowolf timowolf added the documentation Improvements or additions to documentation label Jan 30, 2026
@spike-rabbit spike-rabbit force-pushed the docs/icon-viewer branch 2 times, most recently from 76791c4 to d64fca6 Compare January 30, 2026 13:32
@spike-rabbit spike-rabbit requested a review from timowolf January 30, 2026 13:32
Copy link
Member

@timowolf timowolf left a comment

Choose a reason for hiding this comment

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

Many thanks - LGTM

@github-actions
Copy link

Code Coverage

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants