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

Rendering barcode in mobile phones vertically to avoid breakline. #197

Merged
merged 1 commit into from
Nov 30, 2020

Conversation

mayurkathale
Copy link
Contributor

1. Objective

This PR is to fix issue for display of Tesco Lotus barcode on Order success page. Whenever barcode width is longer than width of screen, it breaks into 2 lines which is not readable by Barcode scanner. To overcome this issue, displaying barcode in mobile phones vertically. Keeping desktop version same.

final

Related information:
Related issue(s): https://omise.atlassian.net/browse/FES-228

2. Description of change

  • Update assets/css/omise-billpayment-print.css, Added media css selector to rotate Barcode div by -90 degree.

3. Quality assurance

🔧 Environments:

i.e.

  • WooCommerce: v4.3.0
  • WordPress: v5.4.2
  • PHP version: 7.3.3
  • Omise plugin version: Omise-WooCommerce 4.3 (optional, in case of submitting a new issue)

✏️ Details:

Try to checkout using tesco lotus barcode payment. See the result on the order success page.

4. Impact of the change

Barcode shouldn't break into 2 lines.

5. Priority of change

Normal

6. Additional Notes

NA

@mayurkathale mayurkathale self-assigned this Nov 30, 2020
@mayurkathale mayurkathale changed the title Rendering barcode in mobile phones vertically to avoid breaklines. Rendering barcode in mobile phones vertically to avoid breakline. Nov 30, 2020
Copy link

@tiponche tiponche left a comment

Choose a reason for hiding this comment

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

:)

@mayurkathale mayurkathale merged commit 372b7ff into master Nov 30, 2020
@mayurkathale mayurkathale deleted the mobile-tescobarcode branch November 30, 2020 08:57
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.

5 participants