Some ID's in preview.html are the same as in page.tpl.php - namely for main-menu and the footer-wrapper etc, this can (not always, can't pinpoint the exact conditions) really screw up the preview... its also invalid markup, not so good eh?

Easy fix, change the ID's in preview.html. This looks like a big patch but really its not, its more of a cleanup which should have really been done in the earlier preview patch, but was overlooked. So in effect this patch does two things:

1. Fix the duplicated invalid ID's
2. Cleanup preview.css (most selectors where way over specified).

I think this is pretty major, it can really mess up the preview and is not tolerable for our default theme.

Comments

tim.plunkett’s picture

Code looks great. Now for some testing. I'll have to first find a way to get them to collide, I guess.

tim.plunkett’s picture

Status: Needs review » Reviewed & tested by the community

Turn off the overlay, use Bartik as the admin theme. Kabloooey. Use this patch. Success!

jensimmons’s picture

+1

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed to HEAD. Thanks.

Status: Fixed » Closed (fixed)

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