Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.I found a pretty weird style attribute at 2CO's review table, which brings the review buttons float over the rest of the table. This can't be avoided, as style hardcoded into table tag. I replaced it with a class, so if someone acually need this, he could add these styles to his theme.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | 458360_2checkout_style.patch | 1010 bytes | Island Usurper |
| #2 | in_garland.png | 53.35 KB | neochief |
| #2 | buggy_in_own_theme.png | 15.31 KB | neochief |
| #2 | in_garland_fixed.png | 34.3 KB | neochief |
| uc_2checkout_weird_style_fix.patch | 535 bytes | neochief | |
Comments
Comment #1
rszrama CreditAttribution: rszrama commentedCan you post a screenshot? This is the same bit of styling we use for the PayPal WPS form and it doesn't seem to cause any troubles. : ?
Comment #2
neochief CreditAttribution: neochief commentedYes, sure. Take a look at the in_garland.png first. As you may see, the weird code ballanced by inside table's margins, which included into garland theme.
But if your theme has no such margins (especialy top one (and you really have them removed, if you want your margins work everywhere)), the table itself became a line (as it is display:inline), so couple of buttons are alligning in a such way that shown in buggy_in_own_theme.png. And the real problem is that you can't do anything to fix it, because this styles are hardcoded into the table.
In other hand, if you remove this styles, everything will still looks great (see in_garland_fixed.png)
Comment #3
rszrama CreditAttribution: rszrama commentedComment #4
stephthegeek CreditAttribution: stephthegeek commentedIf the current inline styles are the ones that seem to work in Garland and some other themes, those should probably be set as the default styles for the new class as well, which can be overridden now in the theme if in a case such as yours, they need to be.
Also, only dashes should be used in css classes/IDs, so 2checkout_review_table should be 2checkout-review-table
Comment #5
Island Usurper CreditAttribution: Island Usurper commentedMy text editor didn't seem to like a class that started with a number, and an id seemed more appropriate to me in this case, so I called it
#two-checkout-review-table.Comment #6
stephthegeek CreditAttribution: stephthegeek commentedDon't have a place to test this, but code changes look good!
Comment #7
Island Usurper CreditAttribution: Island Usurper commentedGood enough for me. Thanks, steph.
Committed.