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.

CommentFileSizeAuthor
#5 458360_2checkout_style.patch1010 bytesIsland Usurper
#2 in_garland.png53.35 KBneochief
#2 buggy_in_own_theme.png15.31 KBneochief
#2 in_garland_fixed.png34.3 KBneochief
uc_2checkout_weird_style_fix.patch535 bytesneochief

Comments

rszrama’s picture

rszrama commented

Can 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. : ?

neochief’s picture

neochief commented
StatusFileSize
new in_garland_fixed.png34.3 KB
new buggy_in_own_theme.png15.31 KB
new in_garland.png53.35 KB

Yes, 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)

rszrama’s picture

rszrama commented
Assigned: Unassigned » rszrama
Issue tags: +ubercamp sprint
stephthegeek’s picture

stephthegeek commented
Assigned: rszrama » Unassigned
Status: Needs review » Needs work

If 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

Island Usurper’s picture

Island Usurper commented
Status: Needs work » Needs review
StatusFileSize
new 458360_2checkout_style.patch1010 bytes

My 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.

stephthegeek’s picture

stephthegeek commented

Don't have a place to test this, but code changes look good!

Island Usurper’s picture

Island Usurper commented
Status: Needs review » Fixed

Good enough for me. Thanks, steph.

Committed.

Status: Fixed » Closed (fixed)
Issue tags: -ubercamp sprint

Automatically closed -- issue fixed for 2 weeks with no activity.

Issue tags: +ubercamp sprint

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.