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

Customer account edit form: additional info block visible #8252

Merged
merged 1 commit into from
Feb 9, 2017
Merged

Customer account edit form: additional info block visible #8252

merged 1 commit into from
Feb 9, 2017

Conversation

Sylvco
Copy link
Contributor

@Sylvco Sylvco commented Jan 24, 2017

Moved the additional info block in the customer account edit form outside of the fieldset which hides with the checkboxes options. It is now visible without having to check one of the checkboxes.

@magento-cicd2
Copy link
Contributor

magento-cicd2 commented Jan 24, 2017

CLA assistant check
All committers have signed the CLA.

@okorshenko
Copy link
Contributor

Hi @Sylvco
Thank you for your contribution to Magento 2 project.
Could you please confirm that this is expected Customer UI? Please see screenshots:
Original UI:
01.
origin
02.
origin2

Proposed UI:
proposed

@Sylvco
Copy link
Contributor Author

Sylvco commented Feb 8, 2017

Yes, this is what i meant.

@okorshenko
Copy link
Contributor

@Sylvco thank you for confirmation. I will take this PR in progress

@mmansoor-magento mmansoor-magento merged commit ae5d782 into magento:develop Feb 9, 2017
@okorshenko
Copy link
Contributor

@Sylvco thank you for your contribution. Your fix is delivered to develop branch

@okorshenko okorshenko added this to the February 2017 milestone Feb 11, 2017
@vrann vrann added the simple label Mar 11, 2017
magento-devops-reposync-svc pushed a commit that referenced this pull request May 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants