Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I must say that most of the admin pages, tables and blocks
looks good for RTL site using with this theme out of the box.
I started to add RTL fixes to the nodes and comments
I'm still having so troubles with menus & bullets
Attached what I've done so far (not much)
And a screenshot of how my experimental Hebrew site look like with the current patch
I'll add some more soon :-)
Comment | File | Size | Author |
---|---|---|---|
#9 | ulproblem.png | 6.34 KB | lameei |
#5 | fourseasons.zip | 31.09 KB | Optalgin |
#2 | style-rtl.zip | 1.04 KB | Optalgin |
style-rtl.zip | 1.21 KB | Optalgin | |
4s_rtl.jpg | 79.75 KB | Optalgin |
Comments
Comment #1
derjochenmeyer CreditAttribution: derjochenmeyer commentedcool, thanks for posting... here is another issue about rtl support ..
http://drupal.org/node/225253
Comment #2
Optalgin CreditAttribution: Optalgin commentedSome updates:
Known Issues:
I also have issues with IE6, right sidebar is printed at bottom of content
but this looks like a problem that has nothing to do with the RTL changes
Comment #3
derjochenmeyer CreditAttribution: derjochenmeyer commentedfor the ie issue just decrease the width of the column by 1 or 2 percent .. in ie-fix.css
Comment #4
Optalgin CreditAttribution: Optalgin commentedThanks for the IE6 tip, I just read that in other page few seconds ago :-)
but now I face another thing I'm not sure how to implement
in page.tpl.php there is this code
I wish to change that file for RTL but I'm not sure how to set a condition
Do you think something like that might work??
Anyway I'll test IE6 hacks soon, I need to find a computer with this browser installed
Thanks for your help
Comment #5
Optalgin CreditAttribution: Optalgin commentedI'm attaching my latest changes of RTL support for this theme
I also added a block region called "above content"
Most of the issues from my above post are still open (TBD)
Comment #6
lameei CreditAttribution: lameei commentedThanks for your efforts. have tested this on IE6?
Comment #7
lameei CreditAttribution: lameei commentedI did a test for IE6 there is major problems. you can use Multiple_IE to install different versions of IE on the same computer. here is thee link: http://tredosoft.com/Multiple_IE
Comment #8
Optalgin CreditAttribution: Optalgin commentedthanks for your comments
Please note that this version is incomplete, I have converted only what I needed
and haven't got the time (yet) to fix all problems
I have a running RTL site using this theme and I have no major problems with IE6,IE7,FF2,FF3
can you be more specific, please describe (or attach screen shot) the problems you encountered
Thanks for the link, this is really useful :-)
Comment #9
lameei CreditAttribution: lameei commentedActually i have paste the contents on ie-fix-rtl into ie-fix so the problems have been solved. but there is one problem in ie7 and it is clear in attached picture. the circles and triangles are not positioned well. this happens in collapsed menus only in IE.
This is another software which you can use to test your site on different versions of ie: http://on10.net/blogs/sarahintampa/IETester-Helps-You-Test-Your-Site/
and here is a link which will take a snapshot of your site on different OS and browsers:
http://browsershots.org/
Comment #10
Optalgin CreditAttribution: Optalgin commentedThere is a problem with the 3rd level of the menu and above
I will try to fix this soon
You are also welcome to try and fix it in style-rtl.css
I think the problem is somewhere in this part
Comment #11
derjochenmeyer CreditAttribution: derjochenmeyer commentedthanks a lot for the work that you are doing on rtl ... i would be happy to include style-rtl.css in a future version of four seasons ...
Comment #12
Optalgin CreditAttribution: Optalgin commentedI am happy to work with this theme.. This is an amazing theme!..
I wish I had more time to dedicate for it :-)
Comment #13
lameei CreditAttribution: lameei commentedI have added the following to fix-is CSS file the problem solved for 3rd level. for the other levels it shuold be the same.
.sidebar ul li ul li ul li ul li {
background-position: 11.8em 0.6em;
}