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

[FIX]payment: improve ui of shop/payment page in website. #26376

Closed

Conversation

pka-odoo
Copy link
Contributor

  • align payment method(Ex.Stripe) form attributes in row.
  • fix align between radio button and payment method name.
  • removed header div from 'billing & shipping'.
  • set badges(Ex.free) and align between badges and delivery method name in choose delivery method.
  • border subtraction from delivery methods div as already card class has border.

Description of the issue/feature this PR addresses:

  1. https://drive.google.com/file/d/1fZje7_eSA9H3E4wJ35jYTkQohU5NkVg1/view
  2. https://www.awesomescreenshot.com/image/3517534/d56b76dbef078ad172f73d976bcbca82

Current behavior before PR:
UI issues in shop/payment page in website.

Desired behavior after PR is merged:
Improved ui of shop/payment page.

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

- align payment method(Ex.Stripe) form attributes in row.
- fix align between radio button and payment method name.
- removed header div from 'billing & shipping'.
- set badges(Ex.free) and align between badges and delivery method name in choose delivery method.
- border subtraction from delivery methods div as already card class has border.
@C3POdoo C3POdoo added the RD research & development, internal work label Aug 14, 2018
@qsm-odoo
Copy link
Contributor

Merged 1c44e53

@qsm-odoo qsm-odoo closed this Aug 20, 2018
@qsm-odoo qsm-odoo deleted the master-payment-layout-broken-issue-pka branch August 20, 2018 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants