ritterim / platform-ui

A CSS framework build all of our apps and websites.

Home Page:https://platformui.com

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Consolidate various active classes into just active

jaime-lynn opened this issue · comments

🔧 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?

@jaime-lynn, @hougasian

Estimates

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.

is this complete?

Closing. These changes are in the latest version.

@tedk13 can you please add an estimate