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.
Bartik is currently missing RTL styling. Just like http://www.arabicalistapart.com/ completely flips the design horizontally from http://www.alistapart.com/, so too should Bartik.
Comment | File | Size | Author |
---|---|---|---|
#42 | layout-rtl.css_.txt | 650 bytes | stephthegeek |
#34 | Bartik RTL | 142.8 KB | stephthegeek |
#33 | bartik_rtl_706862_2.patch | 9.36 KB | stephthegeek |
#31 | bartik_rtl_706862_1.patch | 8.52 KB | stephthegeek |
#29 | bartik_rtl.patch | 13.08 KB | tsi |
Comments
Comment #1
Bojhan CreditAttribution: Bojhan commentedI am willing to help on this, we should have propper support on this.
Comment #2
Bojhan CreditAttribution: Bojhan commentedGiven that, unless Batrik is in core - this is not critical
Comment #3
jensimmons CreditAttribution: jensimmons commentedYes, please. Let's do it. Write the code someone!
Comment #7
JohnAlbinReverting title after comment spam.
Comment #8
Ahmad Abubakr CreditAttribution: Ahmad Abubakr commentedI'm interested in this issue I've been working on Drupal RTL (Arabic) websites for about a year now.
Comment #9
delmarr CreditAttribution: delmarr commentedBartik has issues with rtl and fieldsets, the title of the fieldset floats far to the left off the page. I've attached a screenshot.
Comment #10
willmoy CreditAttribution: willmoy commentedNow we have a bartik project, moving there. Marking as critical because it blocks bartik in core.
Bojhan, ball's in your court if you want it!
Comment #11
yoroy CreditAttribution: yoroy commentedFrom style.css @ arabicalistapart.com:
Is that how Bartik should do it as well? Is there a conditional body class available to hang this RTL layout off of?
Comment #12
jensimmons CreditAttribution: jensimmons commentedre #10: Let's keep this as critical. The whole point is to get Bartik into core.
re #9 — that's a screenshot of Seven in the overlay. Is the problem it's having unique to Bartik? If Garland is the current public-facing theme, doesn't Seven do the same thing? If it's a problem w/ Seven, file the issue over there. If somehow Seven is being affected by Bartik (?), then let's track down what's happening and fix it.
Comment #13
yoroy CreditAttribution: yoroy commentedthere's an epic fieldset fixing issue over here: #676800: Fieldsets break design badly that's not yet fully fixed. revisit this as a seperate issue if needed after that issue is completely fixed.
Comment #14
yoroy CreditAttribution: yoroy commentedTalked this over with johnalbin in irc:
Comment #15
tsi CreditAttribution: tsi commentedI'm on it.
I will be working on the github version and place a patch here, is that the way to go ?
as I understand it I will create :
1. layout-rtl.css
2. style-rtl.css
3. a patch against style.css (probably just to add /* LTR */ comments)
4. and one against layout.css (/* LTR */ comments)
If all is right and I find the time we can expect first results tommorow, this shouldn't be too hard.
Comment #16
emmajane CreditAttribution: emmajane commentedPlease work on the version that's in CVS on d.o. It is up to date.
Comment #17
iamjon CreditAttribution: iamjon commentedHi TSI,
I can help you out motze shabbat if you need just let me know
Comment #18
iamjon CreditAttribution: iamjon commentedAttached are two screenshots of Bartik in it's current form, with a user logged in and logged out.
I have to update the translations on my test site and add a bit more content but here it is,
Comment #19
tsi CreditAttribution: tsi commented@emmajane - I have checked out the version from cvs and it doesn't look as good as the github version.
I still have a lot to learn about cvs so I might not have downloaded the right version.
Attached are two screenshots of Bartik, one from github and one from cvs with the differences marked.
please point me to the right path in cvs.
I can work on it in the next few hours so a quick response will be appreciated.
Comment #20
yoroy CreditAttribution: yoroy commentedTSI: any glitches in the CVS version are bugs we should fix. Do use a CVS checkout of Bartik HEAD please.
Comment #21
JacineAh, this was actually caused by an old sytax error in the #navigation. It's fixed in CVS now. Thank you for pointing it out.
Comment #22
tsi CreditAttribution: tsi commentedAs promised...
Here are the files needed for basic RTL support in Bartik.
This is only about adding the directionality overrides - nothing more, nothing less. I will probably open other issues with a few other things I've noticed that need some love.
in the tar :
1. layout-rtl.css
2. style-rtl.css
3. a patch against style.css with /* LTR */ comments
4. and one against layout.css with /* LTR */ comments
5. flipped comment-arrow-rtl.gif
I tested on a few browsers (FF, Chrome, IE7-8) but NOT thoroughly, IE7 has some issues but I don't think they are RTL related.
If someone have problems setting up an RTL test-site I can attach a few screenshots.
[edit] this is my first work against cvs, I hope it is done right.
Comment #23
yoroy CreditAttribution: yoroy commentedhmm, thanks but not ideal :) A single patch that creates the two new files would be best. Let me try to reroll what you did. Maybe http://www.yoroy.com/2009/create-drupal-patches-aptana-tutorial can be helpful?
Let me try to reroll what you did.
Comment #24
yoroy CreditAttribution: yoroy commentedLets see. This is only converting #22 to a single patch, I did not review nor test the actual styles.
Comment #25
tsi CreditAttribution: tsi commentedStill learning the art of patch work, thanks for the link.
Comment #26
yoroy CreditAttribution: yoroy commentedno problem. status back to needs review
Comment #27
tsi CreditAttribution: tsi commentedYour patch won't make the changes to style.css and layout.css, won't it ?
I think now I've got it right :
Comment #28
tsi CreditAttribution: tsi commenteddon't know why it changed... again...
Comment #29
tsi CreditAttribution: tsi commentedsmall fix
Comment #30
yoroy CreditAttribution: yoroy commentedCorrect, i forgot the changes to the existing stylesheets. You're getting the hang of it! :)
Comment #31
stephthegeek CreditAttribution: stephthegeek commentedRerolled to keep up with HEAD but made no other changes. I spent a couple of hours testing this with some Arabic content and could not find any issues that were related to the theme. My interface translation was only partial and testing was done with a dozen nodes/comments, but the patch looks really solid, nice job!
There's an issue with horizontal scrollbars permanently appearing in RTL in Chrome and Opera. Oddly enough, it's the opposite of the chromium bug reported here (http://code.google.com/p/chromium/issues/detail?id=10533) but the same fix (going from
dir="rtl"
on the<body>
to#page-wrapper { direction: rtl; }
) fixes it, but considering this is consistent with core's html.tpl.php, this doesn't feel like an issue to be addressed in Bartik itself -- just noting the issue.Comment #32
stephthegeek CreditAttribution: stephthegeek commentedWill need updating for #713444: Add a second sidebar
Comment #33
stephthegeek CreditAttribution: stephthegeek commentedSincerely hoping I'm doing this whole patching thing correctly... here's an updated patch for the second sidebar.
Comment #34
stephthegeek CreditAttribution: stephthegeek commentedAnd a screenshot of the layout with two sidebars and some RTL content.
Comment #35
jensimmons CreditAttribution: jensimmons commentedThis looks good to me, with a quick review. I am committing it as-is so that we can keep working on the CSS that is affected by this patch without making this patch out-of-date.
This STILL NEEDS TESTING however. Please do test the Bartik head and make sure that Bartik works happily RTL. If we need additional patches, let's talk about that and make them below.
In other words, committed but not fixed!
Comment #36
JacineWhere is the styles-rtl.css file? :)
Comment #37
iamjon CreditAttribution: iamjon commentedI'll test it out with later today and post some screenshots
Comment #38
Bojhan CreditAttribution: Bojhan commentedpublished on, seems located at the wrong place.
Comment #39
jensimmons CreditAttribution: jensimmons commentedI committed the patch in #33.
I also committed the image that was in #24.
Is looks from reading the patch in #33 that css/layout-rtl.css and css/style-rtl.css didn't make it into the patch. Dang CVS!
http://drupal.org/files/issues/bartik_rtl_706862_2.patch
Could someone either make a new patch, or simply attach those files to this issue, and I'll get them into CVS.
Comment #40
tsi CreditAttribution: tsi commentedThey are in #22 and #24 creates them (but before second sidebar fixes)
Comment #41
jensimmons CreditAttribution: jensimmons commentedAh, there we go! Thanks TSI.
Ok, I just added layout-rtl.css and style-rtl.css to Bartik, and committed that add.
Please do test and see if those two stylesheets need updates for the new second column. I bet they do. Steph if you already made the changes to the CSS in those files, you should be able to patch against the latest Bartik Head successfully. Adding new files to CVS is tricky — now that the files are there, you can modify them much more easily. Just make another patch and your changes will go in.
More feedback?
Comment #42
stephthegeek CreditAttribution: stephthegeek commentedHere's my layout-rtl.css that was updated for the new sidebar. No changes were made in style-rtl.css from the earlier patch. I suspect I just did something wrong with the patch and it didn't include the new files.
Comment #43
jensimmons CreditAttribution: jensimmons commentedCommitted revised layout-rtl.css. Fabulous. Thanks steph and everyone.
So NOW we should have all the code and images for RTL support that has been created to date. Please test and let us know what you discover.
Comment #44
Jeff Burnz CreditAttribution: Jeff Burnz commentedCan we change this to fixed, to take it off the radar, its a bit confusing to have something that's been committed marked as needing review, if issues crop up we can re-open as needing work.
Comment #46
Gábor HojtsyLooks like the Bartik theme as it is in Drupal 8 broke this, see #1979392: Bartik layout broken in RTL.