Closed Bug 80577 Opened 21 years ago Closed 20 years ago
Order of words reversed in Hebrew site on Mac
94.47 KB, image/tiff
123.83 KB, image/tiff
54.47 KB, image/gif
61.56 KB, image/gif
25.04 KB, image/gif
107.31 KB, image/jpeg
I was sent the attached screen shots by firstname.lastname@example.org Each word is displayed correctly, but the order of the words is reversed. For example the Hebrew equivalent of WHAT'S NEW, which should be rendered WEN S'TAHW is rendered S'TAHW WEN
The first thing to try in fixing this is to port the IBMBIDI changes in gfx/src/windows/nsRenderingContextWin.cpp to gfx/src/mac/nsRenderingContextMac.cpp
simon- Please attach nwe screen shot in GIF or JPEG. I somehow cannot view tiff. Also, could you attach the CORRECT screen shot here so I can compare if I write some new code. reassing to ftang
Assignee: nhotta → ftang
mark it as assigned
Status: NEW → ASSIGNED
Attaching new screenshots. For testing I suggest you use the Hebrew testpage that Mimi prepared for Andreas, which has text in Hebrew with images of the expected rendering.
I am confsued. Which screen shots are right, which are wrong ?
WRONG: attachment 34529 [details] -- walla attachment 34530 [details] -- msn RIGHT: attachment 34531 [details] -- walla I didn't attach a RIGHT screenshot of www.msn.co.il, because the content of the site keeps changing.
no time to work on mac arabic/hewbrew . move to future.
Target Milestone: --- → Future
Changing QA contact to email@example.com. Changing component to "Bidi..." and adding intl keyword.
Component: Internationalization → BiDi Hebrew & Arabic
QA Contact: andreasb → giladehven
*** Bug 94633 has been marked as a duplicate of this bug. ***
*** Bug 93665 has been marked as a duplicate of this bug. ***
*** Bug 81943 has been marked as a duplicate of this bug. ***
*** Bug 81020 has been marked as a duplicate of this bug. ***
From dupe: ------- Additional Comments From Simon Montagu 2001-05-18 11:52 ------- What I was trying to say was that the Hebrew in your screenshot has been ordered with base direction left-to-right instead of right-to-left. In Windows when mRightToLeftText is true in nsRenderingContextWin.cpp we call SetTextAlign with TA_RTLREADING, which sets the base direction for the device context's native reordering to right-to-left. You have to find the equivalent API for Mac and call it. ------- Additional Comments From Simon Montagu 2001-05-21 15:51 ------- What I meant was the base (paragraph) direction used when the platform invokes its own implementation of the Bidi algorithm. This is important, because it determines whether a logical text like COPYRIGHT 2000 MICROSOFT is rendered TGHIRYPOC 2000 TFOSORCIM <--- with LTR base direction or TFOSORCIM 2000 THGIRYPOC <--- with RTL base direction In Windows this is set by calling SetTextAlign with the flag TA_RTLREADING (by the way the comment in the source of nsRenderingContextWin.cpp is not synchronized with the code -- I'll file a separate bug on that).
OS: other → Mac System 8.5
*** Bug 96305 has been marked as a duplicate of this bug. ***
Mass-move all BiDi Hebrew and Arabic qa to me, firstname.lastname@example.org. Thank you Gilad for your service to this component, and best of luck to you in the future. Sholom.
QA Contact: giladehven → zach
*** Bug 108331 has been marked as a duplicate of this bug. ***
On system 10, I see this in some places but not others. not sure if this is the same bug? adding attachment
I believe the following three patch should fix most mac os hewbrew problem big endian issue in nsFrame.cpp see latest patch in http://bugzilla.mozilla.org/show_bug.cgi?id=116976 big endian issue in nsBidiImp.cpp http://bugzilla.mozilla.org/attachment.cgi?id=62917&action=view turn off hebrew reordering and do not report hint on mac gfx http://bugzilla.mozilla.org/attachment.cgi?id=62920&action=view
this should be fixed right now. If you see some speical case which are not right. Please file seperate bug for the speical case and assign to smontagu first.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Component: Layout: BiDi Hebrew & Arabic → Layout: Text
QA Contact: zach → layout.fonts-and-text
You need to log in before you can comment on or make changes to this bug.