-
Notifications
You must be signed in to change notification settings - Fork 10
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
length of coupon name causes layout issues #35
Comments
There is a major breach on this pluggin. You can get this cupom code and apply to any payment method. I dont think this should be allowed. I really should search for an alternative to this pluggin. |
@lotuzwine I've thought the same at first. But try it. |
I actually tried and applied the discount using a credit card payment method, which should not have any discount at all. |
I've just tried it (again - it was the first i tried with the new version). While it says "coupon applied" at the messages, it doesn't actually apply it. Try it again and pay attention to the totals. The actual issue is the message "coupon applied" appearing. It should not. But nobody can cheat (not on my installation at least). |
I can confirm that the coupon code "successfully" applies to other payment methods but it gives a discount of zero. Please create a separate thread for this as it is not related to the topic. |
Thread created. |
Does someone was able to solve this problem? I'm having the same problem. |
@claudiosanches why don't you just change the '_' character to '-' ? such a low hanging fruit. if you care, just do it. It has been 3 years... it's time! |
Most themes aren't prepared for super long coupon names. On small resolutions prices disappear off-screen and can't be scrolled to. On mobile, this results in prices / totals not being visible at all without a way to make it visible.
Example: Storefront

Strictly speaking, this is a theme issue and can be solved by custom CSS.
However, also for usability, I'd prefer something that is very short and human readable. E.g. "3% BACS". Using the woocommerce_cart_totals_coupon_label filter maybe?
A quick solution would be to use "-" instead of "_" so the browser can break the line.
The text was updated successfully, but these errors were encountered: