link underlines print too thickly

• Component: Printing
• Status: RESOLVED FIXED

[Trey Matteson]

Go to any page with links.  Observe how the underlines look.
Print the page (Preview works too).
Note the underlines are way too thick.  They don't look good and they do not look they did on the screen.

[Trey Matteson]

Created attachment 3157 [details]
simple test file with one link, useful for debugging

[Trey Matteson]

Created attachment 3158 [details]
cvs attrib output

The funny thing is if you look at the history of the attached method, it looks
to me like John fixed it on 1/11/05 and mjs broke it two days later.  This
could have just been a botched merge.

Removing the unconditional [graphicsContext setShouldAntialias: NO]; does seem
to fix the basic case for me.

[Maciej Stachowiak]

Yes odds are good I botched the merge - when reapplying fix please make sure international text input 
still looks right.

[Trey Matteson]

Created attachment 3187 [details]
proposed patch, per recent email

[Darin Adler]

Comment on attachment 3187 [details]
proposed patch, per recent email

Maciej and I were talking about this on IRC and clearly he was saying "r=me".

[Justin Garcia]

Landing this ... international text input looks OK.

[John Sullivan]

This is also in Radar as 3965011.

[Trey Matteson]

Comment on attachment 3187 [details]
proposed patch, per recent email

Hold off - I think I have a better fix

[Trey Matteson]

Oh well, it's already in, I made a new bug - #4255