Skip to content
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

Update button state classes to follow naming convention. #416

Merged
merged 1 commit into from
Jan 30, 2015
Merged

Update button state classes to follow naming convention. #416

merged 1 commit into from
Jan 30, 2015

Conversation

DFurnes
Copy link
Contributor

@DFurnes DFurnes commented Jan 29, 2015

Changes

  • Update button state classes to follow naming convention.

Note: Loading styles used to use a pseudo-element, and so did not work on <input> elements. This was changed a little ways back when we switched to using SVG for spinners rather than building them using CSS elements.

For review: @DoSomething/front-end

@sbsmith86
Copy link
Contributor

👍

1 similar comment
@weerd
Copy link
Contributor

weerd commented Jan 30, 2015

👍

DFurnes added a commit that referenced this pull request Jan 30, 2015
Update button state classes to follow naming convention.
@DFurnes DFurnes merged commit 9478e47 into DoSomethingArchive:dev Jan 30, 2015
@DFurnes DFurnes deleted the button-state-classes branch April 1, 2015 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants