+
Skip to content
This repository was archived by the owner on Oct 14, 2025. It is now read-only.

Conversation

ynotdraw
Copy link
Collaborator

@ynotdraw ynotdraw commented Sep 24, 2024

🚀 Description

Moving Status Indicator upstream as it's only used in Toasts and has variants specific to one application, allowing Core to focus on shared components.

📋 Checklist

🔬 How to Test

📸 Images/Videos of Functionality

N/A

@ynotdraw ynotdraw self-assigned this Sep 24, 2024
Copy link

changeset-bot bot commented Sep 24, 2024

🦋 Changeset detected

Latest commit: f9049a4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@crowdstrike/glide-core Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor

}
.label {
color: var(--glide-core-text-body-2);
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Explicitly set the colors, otherwise the color of the icon would get inherited by the text.

@ynotdraw ynotdraw force-pushed the remove-status-indicator branch from c2efdf3 to fa2529a Compare September 24, 2024 23:13
@ynotdraw ynotdraw marked this pull request as ready for review September 24, 2024 23:13
'@crowdstrike/glide-core': minor
---

Removed the Status Indicator component.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Say why?

@ynotdraw ynotdraw force-pushed the remove-status-indicator branch from fa2529a to b4a0dd0 Compare September 25, 2024 12:59
Co-authored-by: clintcs <114178960+clintcs@users.noreply.github.com>
@ynotdraw ynotdraw merged commit 7231487 into main Sep 25, 2024
7 checks passed
@ynotdraw ynotdraw deleted the remove-status-indicator branch September 25, 2024 23:15
@github-actions github-actions bot mentioned this pull request Sep 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载