After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 107526 - GtkScrolledWindow needs RTL support
GtkScrolledWindow needs RTL support
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Widget: Other
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks: 76219
 
 
Reported: 2003-03-04 05:35 UTC by Noah Levitt
Modified: 2011-02-04 16:16 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
gtkscrolledwindow.c patch (2.01 KB, patch)
2003-05-25 22:34 UTC, Matthias Clasen
none Details | Review
gtkviewport.c patch (1.57 KB, patch)
2003-05-25 22:35 UTC, Matthias Clasen
none Details | Review
viewport.c patch (1.41 KB, patch)
2003-07-07 22:51 UTC, Matthias Clasen
none Details | Review
Patch for testgtk to reproduce the bug (3.62 KB, patch)
2003-11-28 02:24 UTC, chinen
none Details | Review
Proposed patch - based on CVS 2003/Nov/19 (2.44 KB, patch)
2003-11-28 02:32 UTC, chinen
none Details | Review

Description Noah Levitt 2003-03-04 05:35:59 UTC
1. As jrb mentioned in the announcement email about tree view RTL, a
scrolled window with a horizontal scrollbar expands/contracts/shows/hides
(not sure what to call it) the wrong side of the scrolled window.

2. Vertical scrollbars seem to always show up on the right side. Should be
on the left in RTL, I think.
Comment 1 Matthias Clasen 2003-05-25 22:34:14 UTC
I'll attach two patches here. The first is a patch for
gtkscrolledwindow.c, which fixes issue 2. The second is a patch for
gtkviewport.c which tries to fix issue 1. It is not working perfectly
though, someone will have to investigate why it flickers so badly.
And, of course, other "scrollable" widgets will have to be
individually fixed wrt. issue 1.
Comment 2 Matthias Clasen 2003-05-25 22:34:51 UTC
Created attachment 16837 [details] [review]
gtkscrolledwindow.c patch
Comment 3 Matthias Clasen 2003-05-25 22:35:22 UTC
Created attachment 16838 [details] [review]
gtkviewport.c patch
Comment 4 Noah Levitt 2003-07-07 21:16:49 UTC
FYI, the viewport patch no longer applies successfully to HEAD. :-\
Comment 5 Matthias Clasen 2003-07-07 22:51:18 UTC
Created attachment 18115 [details] [review]
viewport.c patch
Comment 6 chinen 2003-11-21 05:13:08 UTC
Another patch for issue 2 is on the following URL.
It was created from the source in CVS on 2003/Nov/19

http://www-124.ibm.com/developerworks/oss/linux/patches/?patch_id=1191

Thank you,
Comment 7 chinen 2003-11-28 02:24:28 UTC
Created attachment 21873 [details] [review]
Patch for testgtk to reproduce the bug
Comment 8 chinen 2003-11-28 02:30:04 UTC
I'd like to write the steps to reproduce the bug.

1. Apply the testgtk-for_test.20031119.patch to tests/testgtk.c.
   (The patch is the one I attached in last comment.)
   The purpose of this is to change direction of 'scrolled windows'
   dialog at the runtime.
2. Run testgtk in the bidi locale. (e.g. he_IL)
3. Choose item 'scrolled windows'.
   Initial position of the vertical scroll bar is the right side. <--
Problem
4. Change direction of dialog, check position of vertical bar again.
   The vertical scroll bar stays at the right side. <-- Problem
Comment 9 chinen 2003-11-28 02:32:07 UTC
Created attachment 21874 [details] [review]
Proposed patch - based on CVS 2003/Nov/19
Comment 10 chinen 2003-11-28 02:34:45 UTC
The proposed patch is the same one at the above-mentioned URL.
Thanks.
Comment 11 Matthias Clasen 2003-12-05 23:35:55 UTC
I've committed my patches now.