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

Use html5 color input for contribution page widget #12339

Merged
merged 2 commits into from
Jun 20, 2018

Conversation

colemanw
Copy link
Member

@colemanw colemanw commented Jun 19, 2018

Overview

I was just informed that something called HTML5 has been invented. Maybe we should start using it ;)

Before

screenshot 2018-06-19 11 54 35

After

screenshot 2018-06-19 11 54 59

@civibot
Copy link

civibot bot commented Jun 19, 2018

(Standard links)

@eileenmcnaughton
Copy link
Contributor

test this please

@eileenmcnaughton eileenmcnaughton added sig:user interfacing improvement merge ready PR will be merged after a few days if there are no objections labels Jun 19, 2018
@eileenmcnaughton
Copy link
Contributor

I think this is mergeable - but I added merge-ready instead just in case anyone else has any input - I guess my only question is whether there are any compatibility issues - but I guess we are using this already on tags?

@colemanw
Copy link
Member Author

On browsers that don't support html5, there will be no change (they will look like text inputs as before).

@eileenmcnaughton eileenmcnaughton merged commit 4fd4390 into civicrm:master Jun 20, 2018
@eileenmcnaughton eileenmcnaughton deleted the colors branch June 20, 2018 02:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
master merge ready PR will be merged after a few days if there are no objections sig:user interfacing improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants