Bug 24118

Summary: RTL: Truncation behaves differently than IE/FF/Opera
Product: WebKit Reporter: Jeremy Moskovich <playmobil>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, jshin, mitz, ojan, progame+wk, rniwa, xji
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
URL: http://dr-george.co.il
Attachments:
Description Flags
IE/FF/Opera's behavior - left side is truncated.
none
WebKit truncates from the opposite side.
none
Testcase
none
Layout test for rtl truncation .
none
Patch for landing none

Jeremy Moskovich
Reported 2009-02-24 02:13:50 PST
Steps to reproduce: 1. Navigate to http://dr-george.co.il 2. Resize the page so the content doesn't fit horizontally and not the direction the content is truncated from. Expected behavior: FF/IE/Opera truncate the right side of the page. WebKit truncates from the left side. See attached screenshots.
Attachments
IE/FF/Opera's behavior - left side is truncated. (85.23 KB, image/png)
2009-02-24 02:23 PST, Jeremy Moskovich 		no flags
Details
WebKit truncates from the opposite side. (70.67 KB, image/jpeg)
2009-02-24 02:27 PST, Jeremy Moskovich 		no flags
Details
Testcase (1.83 KB, text/html)
2010-10-27 11:55 PDT, Jeremy Moskovich 		no flags
Details
Layout test for rtl truncation . (6.15 KB, patch)
2010-10-28 06:55 PDT, Jeremy Moskovich 		no flags
DetailsFormatted DiffDiff
Patch for landing (74.54 KB, patch)
2010-10-30 22:04 PDT, Jeremy Moskovich 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Jeremy Moskovich
Comment 1 2009-02-24 02:17:49 PST
Filed in the Chromium bug db as: http://crbug.com/7764
Jeremy Moskovich
Comment 2 2009-02-24 02:18:22 PST
Sorry, that should be: http://crbug.com/7763
Jeremy Moskovich
Comment 3 2009-02-24 02:23:33 PST
Created attachment 27911 [details] IE/FF/Opera's behavior - left side is truncated.
Jeremy Moskovich
Comment 4 2009-02-24 02:27:06 PST
Created attachment 27912 [details] WebKit truncates from the opposite side.
mitz
Comment 5 2009-02-24 12:56:11 PST
Looks related to bug 23556 and bug 3352.
Yair Yogev
Comment 6 2009-05-14 03:59:15 PDT
this is not only a mac issue of course
Xiaomei Ji
Comment 7 2010-10-08 15:53:09 PDT
test URL: http://www.dentalspace.co.il
Jeremy Moskovich
Comment 8 2010-10-27 11:55:14 PDT
Created attachment 72065 [details] Testcase Testcase illustrating the issue.
Jeremy Moskovich
Comment 9 2010-10-27 11:57:03 PDT
Looks like this is now fixed on ToT, leaving open for the sole purpose of creating a layout test if this isn't covered by an existing test.
Jeremy Moskovich
Comment 10 2010-10-28 06:55:38 PDT
Created attachment 72184 [details] Layout test for rtl truncation .
WebKit Commit Bot
Comment 11 2010-10-28 07:50:22 PDT
Comment on attachment 72184 [details] Layout test for rtl truncation . Rejecting patch 72184 from commit-queue. Failed to run "['./WebKitTools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=abarth-cq-sl', 'apply-attachment', '--force-clean', '--non-interactive', 72184]" exit_code: 2 Last 500 characters of output: Git diff contains a binary file without the binary data in line: "Binary files /dev/null and b/LayoutTests/platform/mac/fast/block/basic/truncation-rtl-expected.png differ". Be sure to use the --binary flag when invoking "git diff" with diffs containing binary files. at /Users/abarth/git/webkit-queue/WebKitTools/Scripts/VCSUtils.pm line 567, <ARGV> line 59. Failed to run "[u'/Users/abarth/git/webkit-queue/WebKitTools/Scripts/svn-apply', u'--reviewer', u'Ojan Vafai', u'--force']" exit_code: 9 Full output: http://queues.webkit.org/results/4782059
Jeremy Moskovich
Comment 12 2010-10-30 22:04:35 PDT
Created attachment 72450 [details] Patch for landing
WebKit Commit Bot
Comment 13 2010-10-31 19:37:57 PDT
Comment on attachment 72450 [details] Patch for landing Clearing flags on attachment: 72450 Committed r71005: <http://trac.webkit.org/changeset/71005>
WebKit Commit Bot
Comment 14 2010-10-31 19:38:03 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.