-
Notifications
You must be signed in to change notification settings - Fork 1
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
EP-2560-QA-2 #1133
EP-2560-QA-2 #1133
Conversation
@@ -48,17 +48,18 @@ | |||
<h4 ng-if="$ctrl.useV3 !== 'true'" class="panel-title border-bottom-small visible" translate>{{'YOUR_INFORMATION'}}</h4> | |||
<h4 ng-if="$ctrl.useV3 === 'true'" class="panel-title border-bottom-small visible" translate>{{'CONTACT_INFO'}}</h4> | |||
<div class="row"> | |||
<div class="col-sm-4"> | |||
<div ng-class="{'col-sm-4': $ctrl.useV3 !== 'true', 'col-sm-3': $ctrl.useV3 === 'true'}"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
col-sm-6
would be 50% on desktop, which is what I think we would want on v3 (instead of col-sm-3
)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one thing I saw for this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, I would squash commit 4 into commit 2 before merging.
5d0458b
to
015c7fa
Compare
53cf9d1
into
EP-2517-branded-checkout-improvements
Address additional requests made in EP-2517
src/app/branded/step-1/branded-checkout-step-1.tpl.html
move>
line upsrc/common/components/contactInfo/emailField/emailField.tpl.html
change file's indentation to 2 spaces.