Here is what I see now:
Edit Article This is a test story | d7

Here is what you get after the patch:
Edit Article This is a test story | d7

CommentFileSizeAuthor
#4 Windows 7 [Running].jpg35.01 KBbleen
#1 Screen shot 2010-07-31 at 3.17.25 PM.png29.89 KBdrupala
filter.patch951 bytesbleen

Comments

drupala’s picture

drupala commented
StatusFileSize
new Screen shot 2010-07-31 at 3.17.25 PM.png29.89 KB

Is this issue browser issue specific? It looks fine in Firefox without the patch.

bleen’s picture

bleen commented

It's Bartik specific ... the "seven" theme does not have this issue.

malcolm8’s picture

malcolm8 commented

It appears to be fixed in Firefox, but IE7 still shows a pixel space (as well as mismatched widths of the "body" "grippie" and area that says "text format", etc. I'd take a screenshot but I'm using Virtual Box on a Mac laptop.)

What's the protocol for IE? I notice that the rounded corners on tabs have not been recreated yet, for IE.

Is there an outline for theme work that I should be reading? This is literally the first patch I've looked at!

bleen’s picture

bleen commented
StatusFileSize
new Windows 7 [Running].jpg35.01 KB

malcolm8 ... welcome!

a) I'm looking at this using virtual box also and this is what I get. Maybe someone else can confirm one way or the other...
Windows 7 [Running].jpg

b) The rounded corners in the Bartik theme are considered a progressive enhancement. There is no plan to create them using images for IE

(FYI ... I take screenshots using by using skitch and making my virtual box not "full screen")

aspilicious’s picture

aspilicious commented
Status: Needs review » Reviewed & tested by the community

I confirm bleen's screenshots.
This is good to go.

Also tested chrome and opera.

malcolm8’s picture

malcolm8 commented

Thanks, Bleen18. I will try Skitch. : )

The mismatched widths of the divs in IE: a separate issue/issue queue, right?

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

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