Consolidate various active classes into just active
jaime-lynn opened this issue · comments
jaime-lynn commented
🔧 Enhancement Request
Right now we have various classes such as drawer--active
and modal--active
to indicate an active state. After discussion, these separate classes should be removed and just use the active
class.
active
is considered a concept and it should be the same to toggle it across out components.
Who is asking for the feature?
Estimates
Ted Krueger commented
Not that it matters, this issue has my steadfast approval. On a more serious note, I feel like this should be a higher priority than a p3
.
Kevin Hougasian commented
is this complete?
Ted Krueger commented
Closing. These changes are in the latest version.
mackenziegates commented
@tedk13 can you please add an estimate