|Summary:||link underlines print too thickly|
|Product:||WebKit||Reporter:||Trey Matteson <trey>|
|Component:||Printing||Assignee:||Trey Matteson <trey>|
|OS:||OS X 10.4|
Description Trey Matteson 2005-07-30 14:34:04 PDT
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.
Comment 1 Trey Matteson 2005-07-30 14:34:48 PDT
Created attachment 3157 [details] simple test file with one link, useful for debugging
Comment 2 Trey Matteson 2005-07-30 14:41:30 PDT
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.
Comment 3 Maciej Stachowiak 2005-07-31 20:56:06 PDT
Yes odds are good I botched the merge - when reapplying fix please make sure international text input still looks right.
Comment 4 Trey Matteson 2005-07-31 22:05:50 PDT
Created attachment 3187 [details] proposed patch, per recent email
Comment 5 Darin Adler 2005-08-01 00:02:16 PDT
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".
Comment 6 Justin Garcia 2005-08-01 09:35:22 PDT
Landing this ... international text input looks OK.
Comment 7 John Sullivan 2005-08-01 09:55:45 PDT
This is also in Radar as 3965011.
Comment 8 Trey Matteson 2005-08-01 11:46:55 PDT
Comment on attachment 3187 [details] proposed patch, per recent email Hold off - I think I have a better fix
Comment 9 Trey Matteson 2005-08-01 11:52:52 PDT
Oh well, it's already in, I made a new bug - #4255