Closed Bug 206089 Opened 22 years ago Closed 19 years ago

Tiny layout problem in Arabic or Hebrew (Right to Left) language

Categories

(Core :: Layout: Block and Inline, defect)

x86
All
defect
Not set
minor

Tracking

()

RESOLVED INVALID

People

(Reporter: holger.schmeken, Unassigned)

References

(Blocks 2 open bugs, )

Details

(Keywords: testcase)

Attachments

(2 files)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.4b) Gecko/20030507
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.4b) Gecko/20030507

After login in Arabic you see two frames, the left frame is ok, but the right
frame has a tiny problem with a list of LI entries. You can see that the point
of the LI entry is not in the correct position - it is slightly outside of the
table. Please relogin in English and you will see what I mean.

Maybe it is my fault since it is my (beta) isapi extension that you are working
with. It is just a test environment - so the content or design is not important.
The IE 6.0 seems to solve this problem by simply removing the <li> points in
right to left view (hey, great solution ;-).

Regards for your work
 Holger Schmeken, Softwaremanagement.org


Reproducible: Always

Steps to Reproduce:
Attached file Testcase (deleted) —
Confirming bug, 2003-05-16-22 trunk Linux
Assignee: roc+moz → block-and-inline
Severity: trivial → minor
Status: UNCONFIRMED → NEW
Component: Layout: View Rendering → Layout: Block & Inline
Ever confirmed: true
Keywords: testcase
OS: Windows XP → All
Depends on: 74880
There may be additional problem which bug 74880 doesn't regard to:
|remainingWidth == 0| :
http://lxr.mozilla.org/seamonkey/source/layout/html/base/src/nsLineLayout.cpp#2941

This does happen with look and feel I'm currently using. I have a patch for this
problem, but it doesn't make much sense standalone, without fixing bug 74880.
The patch for bug 140611 seems to partly solve this, but we need a clearer
description of the expected and actual results.
Blocks: 137995
Blocks: Persian
Attached file LTR testcase (deleted) —
Note that it's not an RTL bug.

The problem is when you specify a padding for a list (say 10px), mozilla overrides UA padding (default 40px left padding in LTR, and right padding in RTL) and you will have a 10px overall padding, instead of adding them and have a 50px overall padding.

IE adds up user padding with UA padding (at least for lists). Konqueror overrides UA padding with user padding for LTR, and adds them up for RTL!

I will prepare a patch which fixes nsRuleNode::ComputePaddingData() in such a way that it adds 'padding_start_value' and 'padding_left/right_value'.
In fact it is not a bug at all!

W3c explicitly says that users styles override default styles, so there won't be any adding. (http://lists.w3.org/Archives/Public/www-style/1997Nov/0015.html)

About IE and konquerer I was wrong! They have similar behavior. The only difference is that in mozilla the space before bullet is considered by padding but in the w3c sample style it is considered by margin. (http://www.w3.org/TR/REC-CSS2/sample.html)

Mozilla has intentionally chosen padding instead of margin (http://groups.google.com/group/comp.infosystems.www.authoring.stylesheets/browse_thread/thread/1c5fb5936a7845a5?hl=de&lr=&safe=off&ic=1&seekm=slrn92ai53.uuf.dbaron%40is04.fas.harvard.edu#p), so it's not a bug at all!
Yes, this is a duplicate of some other bug that was also INVALID. Unfortunately I can't remember the number, but it doesn't matter.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: