Skip to content

Conversation

@spadgett
Copy link
Member

@spadgett spadgett commented Feb 1, 2017

Disable capitalization, autocorrect, and spell checking for mobile users on the following inputs:

  • Image name field for Add to Project -> Deploy Image
  • Keyword filter in the events directive
  • Name field when editing membership
  • Name filter on monitoring page

@spadgett spadgett requested a review from jwforres February 1, 2017 13:49
@spadgett
Copy link
Member Author

spadgett commented Feb 1, 2017

@jwforres Hold, I'm seeing a few other inputs we should fix as well.

Disable capitalization, autocorrect, and spell checking for mobile users
on the following inputs:

* Image name field for Add to Project -> Deploy Image
* Keyword filter in the events directive
* Name field when editing membership
* Name filter on monitoring page
@spadgett spadgett force-pushed the dont-capitalize-image-name-field branch from 29e8dd2 to d5b1a32 Compare February 1, 2017 14:17
@spadgett spadgett changed the title Turn off capitalization and autocorrect for image name Turn off capitalization and autocorrect for some inputs Feb 1, 2017
@spadgett
Copy link
Member Author

spadgett commented Feb 1, 2017

@jwforres updated

@jwforres
Copy link
Member

jwforres commented Feb 1, 2017

[merge]

@openshift-bot
Copy link

Evaluated for origin web console merge up to d5b1a32

@openshift-bot
Copy link

openshift-bot commented Feb 1, 2017

Origin Web Console Merge Results: SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_origin_web_console/1001/) (Base Commit: 25aa8ff)

@spadgett spadgett force-pushed the dont-capitalize-image-name-field branch from 1665ef1 to d5b1a32 Compare February 1, 2017 14:41
@openshift-bot openshift-bot merged commit 5fe0d17 into openshift:master Feb 1, 2017
@spadgett spadgett deleted the dont-capitalize-image-name-field branch February 1, 2017 16:07
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.

3 participants