filamentphp / filament

A collection of beautiful full-stack components for Laravel. The perfect starting point for your next app. Using Livewire, Alpine.js and Tailwind CSS.
https://filamentphp.com
MIT License
19.18k stars 2.95k forks source link

Are `primary`, `info`, `danger`, `warning` or `success` colors? #13202

Closed bilogic closed 5 months ago

bilogic commented 5 months ago

Package

filament/filament

Package Version

3.2.88

Laravel Version

10.48.12

Livewire Version

3.5.0

PHP Version

8.2.18

Problem description

https://filamentphp.com/docs/3.x/support/blade-components/button#changing-the-color-of-a-button

primary, info, danger, warning and success are technically not colors. Only gray is a color.

Expected behavior

Instead of ->color(...), it would be more appropriate to call it ->tone(), ->theme() or something else

Steps to reproduce

N/A

Reproduction repository

https://github.com/bilogic/filament-issue

Relevant log output

No response

Donate 💰 to fund this issue

Fund with Polar

github-actions[bot] commented 5 months ago

Hey @bilogic! We're sorry to hear that you've hit this issue. 💛

However, it doesn't look like you've provided much information on how to replicate the issue. Please edit your original post with clear steps we need to take.

bilogic commented 5 months ago

At this rate of closure, it's impossible to give good feedback.

danharrin commented 5 months ago

You submitted a bug report for what is basically a feature request or suggestion, which we ask for in Discussions