Summary: | Text overflow ellipsis wrong color when using webkit-text-fill-color | ||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Luke Alonso <luke> | ||||||||||||||||||||||||
Component: | Layout and Rendering | Assignee: | Rob Buis <rwlbuis> | ||||||||||||||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||||||||||||||
Severity: | Normal | CC: | dbates, dglazkov, eric, luke, ojan, rwlbuis, webkit.review.bot | ||||||||||||||||||||||||
Priority: | P2 | ||||||||||||||||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||||||||||||||||
Hardware: | Mac (Intel) | ||||||||||||||||||||||||||
OS: | OS X 10.6 | ||||||||||||||||||||||||||
Attachments: |
|
Description
Luke Alonso
2011-02-20 18:50:41 PST
Comment on attachment 83114 [details] A repro case ><html> ><body> ><div style="width: 225px; font-size: 24px; overflow: hidden; text-overflow: ellipsis; -webkit-text-fill-color: red; white-space: nowrap;"> >Wrong - This is a long string, which is clipped, but the ellipsis is the wrong color! ></div> ><div style="width: 225px; font-size: 24px; overflow: hidden; text-overflow: ellipsis; color: blue; white-space: nowrap;"> >Right - This is a long string, which is clipped, and the ellipsis is the right color! ></div ></body> ></html> Created attachment 146679 [details]
Patch
Comment on attachment 146679 [details]
Patch
Just testing for now.
Attachment 146679 [details] did not pass style-queue:
Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/WebCore/ChangeLog', u'Source/WebCor..." exit_code: 1
Source/WebCore/ChangeLog:8: You should remove the 'No new tests' and either add and list tests, or explain why no new tests were possible. [changelog/nonewtests] [5]
Total errors found: 1 in 2 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 146679 [details] Patch Attachment 146679 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/12935203 New failing tests: fast/css/text-overflow-ellipsis-strict.html fast/css/text-overflow-ellipsis.html Created attachment 146692 [details]
Archive of layout-test-results from ec2-cr-linux-04
The attached test failures were seen while running run-webkit-tests on the chromium-ews.
Bot: ec2-cr-linux-04 Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'> Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Created attachment 146711 [details]
Patch
Comment on attachment 146711 [details] Patch Attachment 146711 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/12919820 New failing tests: fast/css/text-overflow-ellipsis-color.html Created attachment 146718 [details]
Archive of layout-test-results from ec2-cr-linux-04
The attached test failures were seen while running run-webkit-tests on the chromium-ews.
Bot: ec2-cr-linux-04 Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'> Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Created attachment 146893 [details]
Patch
Comment on attachment 146893 [details] Patch Attachment 146893 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/12948247 New failing tests: fast/css/text-overflow-ellipsis-color.html Created attachment 146924 [details]
Archive of layout-test-results from ec2-cr-linux-03
The attached test failures were seen while running run-webkit-tests on the chromium-ews.
Bot: ec2-cr-linux-03 Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'> Platform: Linux-2.6.35-28-virtual-x86_64-with-Ubuntu-10.10-maverick
Created attachment 174822 [details]
Patch
Comment on attachment 174822 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=174822&action=review > LayoutTests/ChangeLog:12 > + * platform/mac/fast/css/text-overflow-ellipsis-color-expected.png: Added. > + * platform/mac/fast/css/text-overflow-ellipsis-color-expected.txt: Added. This should be done with a reference test, not a Mac-only pixel test. Comment on attachment 174822 [details] Patch Attachment 174822 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/14868639 New failing tests: fast/css/text-overflow-ellipsis-color.html Created attachment 174828 [details]
Patch
Comment on attachment 174828 [details] Patch Attachment 174828 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/14875456 New failing tests: fast/css/text-overflow-ellipsis-color.html Created attachment 175003 [details]
Patch
(In reply to comment #18) > Created an attachment (id=175003) [details] > Patch I updated the patch to make it clear that part of the sentence should get clipped and replaced by ellipsis. If I am lucky cr-linux will stop complaining... Comment on attachment 175003 [details] Patch Attachment 175003 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/14906296 New failing tests: fast/css/text-overflow-ellipsis-color.html Created attachment 175110 [details]
Patch
(In reply to comment #21) > Created an attachment (id=175110) [details] > Patch It seems like Dirk's intuition was right, i.e. using Ahem, thanks! I noticed that we also don't support -webkit-text-stroke, -webkit-text-stroke-color, and -webkit-text-stroke-width for an ellipsis. Hi Daniel, (In reply to comment #23) > I noticed that we also don't support -webkit-text-stroke, -webkit-text-stroke-color, and -webkit-text-stroke-width for an ellipsis. Right, but I assume that it is not needed for the ellipsis/dots. I may check against other implementations later... (In reply to comment #24) > Hi Daniel, > > (In reply to comment #23) > > I noticed that we also don't support -webkit-text-stroke, -webkit-text-stroke-color, and -webkit-text-stroke-width for an ellipsis. > > Right, but I assume that it is not needed for the ellipsis/dots. Can you elaborate on your reasoning on why it's unnecessary to support stroke styles for an ellipsis? > I may check against other implementations later... As far as I can tell, -webkit-text-{fill-color, stroke, stroke-color, width} are proprietary WebKit extensions. (In reply to comment #25) > [...] > As far as I can tell, -webkit-text-{fill-color, stroke, stroke-color, width} are proprietary WebKit extensions. s/width/stroke-width (In reply to comment #25) > (In reply to comment #24) > > Hi Daniel, > > > > (In reply to comment #23) > > > I noticed that we also don't support -webkit-text-stroke, -webkit-text-stroke-color, and -webkit-text-stroke-width for an ellipsis. > > > > Right, but I assume that it is not needed for the ellipsis/dots. > > Can you elaborate on your reasoning on why it's unnecessary to support stroke styles for an ellipsis? This is my personal opinion, based on how I imagine it would look. Like described below I can't turn this on for FF, I tried -moz-text-stroke-color and -moz-text-stroke-width. > > I may check against other implementations later... > > As far as I can tell, -webkit-text-{fill-color, stroke, stroke-color, width} are proprietary WebKit extensions. Yeah, from experiments there is nothing equivalent in FF, so that does not help. Given all that, and that I can't see anything about this in the spec, I think it is best to just stick to this patch, and if needed open a bug for the -webkit-text-stroke thing. Comment on attachment 175110 [details]
Patch
r=me
Committed r144542: <http://trac.webkit.org/changeset/144542> |