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

Switching to "color" input type form to change projects color #10

Merged
merged 3 commits into from
Nov 5, 2018

Conversation

teodorg
Copy link
Contributor

@teodorg teodorg commented Mar 19, 2018

"Upgrading" the form for handling project colors to the HTML5 form type "color". In addition to making the code HTML5-friendly, we also get rid of some Javascript code and a solid amount of concats in the entire codebase that add the "#" symbol to all background styles. Instead the html color is saved in the database directly with the "#" in front of it.

teodorg added 2 commits March 19, 2018 22:15
…pe "color". In addition to making the code HTML5-friendly, we also get rid of some Javascript code and a solid amount of concats in the entire codebase that add the "#" symbol to all background styles. Instead the html color is saved in the database directly with the "#" in front of it.
…rch forms. Looks better and makes the system somehow more comfortable to work with.
@teodorg
Copy link
Contributor Author

teodorg commented Mar 22, 2018

Added the "autofocus" attribute to the first fields of forms and search forms. Looks nicer and makes the system somehow more comfortable to work with.

@teodorg
Copy link
Contributor Author

teodorg commented Mar 22, 2018

Changed the search forms from "text" to "search" type.

@benletchford
Copy link
Contributor

benletchford commented Apr 26, 2018

Do we still need dotProject/modules/public/color_selector.php after this?

@teodorg
Copy link
Contributor Author

teodorg commented Apr 26, 2018

Hmmm, looks like - no, we do not.

@teodorg
Copy link
Contributor Author

teodorg commented May 30, 2018

So, are these changes going to be merged into the devel branch or rejected? Any suggestions on the commits ?

@ajdonnison ajdonnison merged commit 0755668 into dotproject:devel Nov 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants