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.
When using this module in an RTL site, the module list appears "compressed" because it leaves 240 pixels on the left, although the tabs are displayed on the right.
collapsed/expanded icons are also LTR only.
Comment | File | Size | Author |
---|---|---|---|
#13 | Screenshot 2024-04-01 at 11.09.33 AM.png | 212.04 KB | smustgrave |
Issue fork module_filter-2161755
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
- 2161755-module-page-is changes, plain diff MR !17
Comments
Comment #1
daneshjoo CreditAttribution: daneshjoo commentedI add this line but it dont work goodly.
Comment #2
hejazee CreditAttribution: hejazee as a volunteer commentedThis patch fixed this problem and a few other RTL related bugs.
Please review it
Note:
You should also download the attached image (collapsed-rtl.png)
and place it under the images/ folder
Comment #3
hejazee CreditAttribution: hejazee as a volunteer commentedModule version numbers should be LTR (new option in 7.x-2.x-dev)
Comment #4
Yago Elias CreditAttribution: Yago Elias as a volunteer and at CI&T commentedHello @hejazee your patch looks good. Just reorded to alphabetic order your rules.
Comment #5
andrey.troeglazov CreditAttribution: andrey.troeglazov at DrupalJedi commentedComment #6
andrey.troeglazov CreditAttribution: andrey.troeglazov at DrupalJedi commentedComment #7
jonathan1055 CreditAttribution: jonathan1055 as a volunteer commentedThis will need to be fixed in 4.x first.
Just for info, csslint reports this problem:
Comment #10
smustgrave CreditAttribution: smustgrave at Mobomo commentedNot the best css person.
Other option we get rid of the clear button and just rely on type="search" x
Comment #11
smustgrave CreditAttribution: smustgrave at Mobomo commentedShould make sure to remove RTL
for example https://www.drupal.org/project/drupal/issues/3294001
Comment #13
smustgrave CreditAttribution: smustgrave at Mobomo commentedWith the latest changes that have gone in the last year I tested again and am not noticing any issue anymore.
See screenshot.
If anyone is still experiencing this though please reopen.