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.How to reproduce:
- On first instalation step choice "Russian" as default site language;
- Use standart profile;
- Create one Article and add several comments to this article;
- Add new RTL language, for example "Arabic";
- Set "Arabic" language as default language.
| Comment | File | Size | Author |
|---|---|---|---|
| #15 | Screen Shot 2016-04-15 at 14.07.36.png | 210.89 KB | emma.maria |
| #15 | Screen Shot 2016-04-15 at 14.06.08.png | 151.06 KB | emma.maria |
| #9 | seven.png | 168.99 KB | skippednote |
| #9 | bartik.png | 178.2 KB | skippednote |
| #7 | seven-rtl-ru-ar.png | 65.71 KB | zestagio |
Comments
Comment #1
zestagio CreditAttribution: zestagio commented#2031883-56: Markup for: comment.html.twig
Comment #2
zestagio CreditAttribution: zestagio commentedComment #3
andypostComment #4
andypostmaybe comments saved before language change are rendered in previous language....
Comment #5
zestagio CreditAttribution: zestagio commentedI did two tests.
Case 1:
Install site with "English" as default site language;
Use standart profile;
Create one Article and add several comments to this article;
Add new RTL language, for example "Arabic";
Set "Arabic" language as default language;
In this case styles are ok
Case 2:
Install site with "Russian" as default site language;
Use standart profile;
Create one Article and add several comments to this article;
Add new RTL language, for example "Arabic";
Set "Arabic" language as default language;
In this case styles are broken
Comment #6
star-szrScreenshots look like they are from Bartik, can you reproduce with any other theme? Sounds like it may be Bartik specific.
Comment #7
zestagio CreditAttribution: zestagio commentedI did test for seven theme.
In case 1 styles for comment's links are broken (seven-rtl-en-ar.png).
In case 2 styles for comments are broken (seven-rtl-ru-ar.png).
Comment #8
star-szrThanks @zestagio.
Comment #9
skippednote CreditAttribution: skippednote at Axelerant commentedI tried both the cases on latest beta and it seems to be working fine with both Bartik and Seven.
Comment #10
zestagio CreditAttribution: zestagio commentedI tested latest beta too. No ok for me. Tested on Windows 7 (Firefox 37.0.2, Google chrome 42.0.2311.135 m) and Ubuntu 12.04 (Firefox 37.0.2, Chromium 37.0.2062.120). Reproduced on all browsers
Comment #11
star-szrNeeds review is for when we have a patch. Thanks for the testing folks!
Comment #12
dbjpanda CreditAttribution: dbjpanda commentedScreenshots are there already.
Comment #13
emma.mariaComment #15
emma.mariaUpdated screenshots were needed as the last were a year ago. I can no longer replicate this issue in any theme - Seven or Bartik.
Comment #16
vprocessor CreditAttribution: vprocessor at Skilld commentedHi guys, I checked this on simplytest.me this steps https://www.drupal.org/node/2480497#comment-9878663
case 2
can't reproduce, all is ok