-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
Terms and Conditions label doesn't fit longer texts #24060
Comments
Hi @DanieliMi. Thank you for your report.
Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:
For more details, please, review the Magento Contributor Assistant documentation. @DanieliMi do you confirm that you were able to reproduce the issue on vanilla Magento instance following steps to reproduce?
|
Hi @engcom-Bravo. Thank you for working on this issue.
|
@magento give me 2.3-develop instance |
Hi @DanieliMi. Thank you for your request. I'm working on Magento 2.3-develop instance for you |
Hi @DanieliMi, here is your Magento instance. |
I can verify the issue on the vanilla instance. |
You could try adding the following style to the checkout agreement button:
|
@tprocter-ldg Awesome idea, but unfortunately not IE11 compatible. https://caniuse.com/#search=display%20contents |
✅ Confirmed by @engcom-Bravo Issue Available: @engcom-Bravo, You will be automatically unassigned. Contributors/Maintainers can claim this issue to continue. To reclaim and continue work, reassign the ticket to yourself. |
Another solution would be: .checkout-agreements-block {
.checkout-agreements {
margin-bottom: @indent__base;
&.fieldset {
> .field.choice{
input[type="checkbox"] {
vertical-align: top;
}
label.label {
display: inline-block;
max-width: 90%;
}
}
}
}
.action-show {
&:extend(.abs-action-button-as-link all);
vertical-align: baseline;
max-width: 90%
}
}
} But this is really not a nice and clean solution just setting the
Is a really nice solution if there wouldn't be the lack of supported browsers. We tried it with |
@magento-engcom-team Can the internal ticket status of MC-19030 be viewed in any way? Is there an estimate for the fix? |
Hi @edenduong. Thank you for working on this issue.
|
@magento i am working on it. |
Hi @webkul-ramakant. Thank you for working on this issue.
|
✅ Confirmed by @engcom-Alfa Issue Available: @engcom-Alfa, You will be automatically unassigned. Contributors/Maintainers can claim this issue to continue. To reclaim and continue work, reassign the ticket to yourself. |
Hi @DanieliMi. Thank you for your report. The fix will be available with the upcoming 2.4.1 release. |
Preconditions (*)
Summary
When creating Terms and Conditions with a longer checkbox text the label and required asterisk will break into new lines. I think this is not expected behavior and looks odd.
Steps to reproduce (*)
1.1. Condition Name: Any
1.2. Status: Enabled
1.3. Show Content As: Text
1.4. Applied: Manually
1.5. Checkbox Text: A text (>= 114 chars)
1.6. Content: Any
Expected result (*)
I expect the Terms and Conditions checkbox to appear and being displayed nice.

Actual result (*)
The text is too long and due to that the label is breaking into a new line. Same with the required asterisk.

The text was updated successfully, but these errors were encountered: