Skip to content
This repository is currently being migrated. It's locked while the migration is in progress.

fix: update calendar_today category and remove event_available icon - #6170

Merged
danbrady merged 2 commits into
mainfrom
db/update-icon-category-2816
Apr 23, 2026
Merged

fix: update calendar_today category and remove event_available icon#6170
danbrady merged 2 commits into
mainfrom
db/update-icon-category-2816

Conversation

@danbrady

@danbrady danbrady commented Apr 23, 2026

Copy link
Copy Markdown
Contributor

This pull request updates the icon metadata in src/_data/icons.yml to improve categorization and remove unused entries. The most important changes are:

Icon categorization improvements:

  • The calendar_today icon now includes the Profile category in addition to Appointments and DS Components, making its categorization more accurate.

Cleanup of unused icon entries:

  • The event_available icon entry, previously categorized under Profile, has been removed in documentation, because calendar_today should be used instead.

Note: event_available was NOT removed from the icon spritesheet as it's currently still in use.

Related Issue

Closes #2816

Preview Environment Links

Open Preview Environment

Copilot AI review requested due to automatic review settings April 23, 2026 14:41
@danbrady danbrady self-assigned this Apr 23, 2026
@danbrady
danbrady marked this pull request as ready for review April 23, 2026 14:44
@danbrady
danbrady requested a review from a team as a code owner April 23, 2026 14:44
@danbrady
danbrady requested a review from a team April 23, 2026 14:53

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 icon metadata used by the design system site so calendar_today is discoverable under the Profile category and removes the event_available icon entry.

Changes:

  • Add Profile to the calendar_today icon’s category list.
  • Remove the event_available icon entry from the icon catalog.

@jeana-adhoc jeana-adhoc 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.

Looks good to me.

I wonder if we'll need to figure out a deprecation path to get the icon out of the sprite sheet eventually.

@danbrady
danbrady merged commit dc40710 into main Apr 23, 2026
5 checks passed
@danbrady
danbrady deleted the db/update-icon-category-2816 branch April 23, 2026 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[icon removal: event_available]

3 participants