Skip to content

LEARNER-3469 - More registration form cleanup. - #16829

Merged
dianakhuang merged 1 commit into
masterfrom
diana/registration-form-improvements
Dec 8, 2017
Merged

LEARNER-3469 - More registration form cleanup.#16829
dianakhuang merged 1 commit into
masterfrom
diana/registration-form-improvements

Conversation

@dianakhuang

@dianakhuang dianakhuang commented Dec 7, 2017

Copy link
Copy Markdown
Contributor

https://openedx.atlassian.net/browse/LEARNER-3469

Sandbox: https://dianakhuang.sandbox.edx.org/register

  1. Added the 'Create an Account' title when the 3rd party auth header isn't shown
  2. Modified the terms of service link to behave like the honor code link
  3. Ensured that both the TOS checkbox and the Honor Code checkbox are below the optional fields toggle.

@dianakhuang
dianakhuang force-pushed the diana/registration-form-improvements branch 2 times, most recently from 90c8793 to e849294 Compare December 7, 2017 21:02
@AlasdairSwan

AlasdairSwan commented Dec 8, 2017

Copy link
Copy Markdown
Contributor

I'm seeing label overlapping content when the browser pre-fills fields on your sandbox
dianakhuang sandbox edx org_register

@AlasdairSwan

Copy link
Copy Markdown
Contributor

I'm actually seeing it in production too (with an added error)
courses edx org_register

@AlasdairSwan

Copy link
Copy Markdown
Contributor

@dianakhuang is there already a bug ticket to resolve the pre-filled overlap issue? If it's a quick fix maybe you could add that to this PR?

@dianakhuang

Copy link
Copy Markdown
Contributor Author

@AlasdairSwan hmm, it seems as though the autocomplete might be behaving wonkily. I will take a look at that.

@dianakhuang
dianakhuang force-pushed the diana/registration-form-improvements branch from e849294 to 0ecc055 Compare December 8, 2017 14:39
@dianakhuang

Copy link
Copy Markdown
Contributor Author

@AlasdairSwan fix for the autocomplete issue has been included.

@AlasdairSwan AlasdairSwan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@dianakhuang
dianakhuang merged commit 8752480 into master Dec 8, 2017
@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production on Monday, December 11, 2017.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

EdX Release Notice: This PR has been deployed to the production environment.

@jdmulloy
jdmulloy deleted the diana/registration-form-improvements branch April 5, 2018 18:43
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