Skip to content

fix: replaced pill with badge #2129

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

HarshMN2345
Copy link
Member

What does this PR do?

fix: replaced pill with badge in containerHeader

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

yes

@coolify-appwrite-org
Copy link

coolify-appwrite-org bot commented Jul 17, 2025

The preview deployment is ready. 🟢

Open Preview | Open Build Logs

Last updated at: 2025-07-17 12:58:34 CET

@HarshMN2345 HarshMN2345 self-assigned this Jul 17, 2025
@coolify-appwrite-org
Copy link

coolify-appwrite-org bot commented Jul 17, 2025

The preview deployment is ready. 🟢

Open Preview | Open Build Logs

Last updated at: 2025-07-17 12:58:35 CET

@HarshMN2345 HarshMN2345 requested a review from ItzNotABug July 17, 2025 12:55
color="pink-2"
on:click={() => (showDropdown = !showDropdown)}
style="cursor:pointer;"
content={`ⓘ ${total}/${limit} created`} />
Copy link
Member

Choose a reason for hiding this comment

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

we can remove the here. Badge accepts a start slot so we can add an Icon component.

@ItzNotABug
Copy link
Member

ItzNotABug commented Jul 17, 2025

@HarshMN2345
Also, lets remove all the old Pill component usages and replace them with the Badge component across console.

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