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.Problem/Motivation
Spacing issue between language menu items in RTL
Steps to reproduce
Select any RTL language and you will see spacing issue between language menu items.
Proposed resolution
Add proper spacing between items.
Remaining tasks
User interface changes
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | Screenshot Before.png | 16.81 KB | Rinku Jacob 13 |
| #6 | Screenshot After.png | 16.75 KB | Rinku Jacob 13 |
| #5 | Screenshot 2023-02-22 at 9.12.39 PM.png | 243.7 KB | Gauravvvv |
| #4 | before and after.png | 136.19 KB | DeepaliJ |
| Screenshot 2023-02-22 at 12.00.07 PM.png | 202.29 KB | Gauravvvv |
Issue fork drupal-3343612
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:
- 3343612-umami-demo-spacing
changes, plain diff MR !3510
1 hidden branch
Comments
Comment #3
Gauravvvv CreditAttribution: Gauravvvv at Axelerant for Drupal India Association commentedComment #4
DeepaliJ CreditAttribution: DeepaliJ at Salsa Digital commentedAble to reproduce thee issue using the steps in IS.
Tested and applied MR #2 on Umami demo ve 10.1.x-dev.
The patch applied cleanly but issue did not get resolve after applying the patch.
Refer to the attached screenshot.
Moving to needs work
Comment #5
Gauravvvv CreditAttribution: Gauravvvv at Axelerant for Drupal India Association commentedplease check carefully. here is after patch screenshot.
Comment #6
Rinku Jacob 13 CreditAttribution: Rinku Jacob 13 at Srijan | A Material+ Company for Drupal India Association commentedHi @Gauravvv , Reviewed your MR, and i got the solution. @DeepaliJ, can you please check once after uncheck the Aggregate CSS files option in performance page then clear the cache . I think you will get the solutions afterthat . Thanks @Gauravvv. Adding Screenshots for the reference. Need RTBC +1.
Comment #7
ameymudras CreditAttribution: ameymudras at Salsa Digital commentedTested on 10.1.x
The issue summary is clear and steps have been provided
Was able to reproduce the issue
Used the MR to fix the issue
Did a code review and no issues were identified
Screenshots have been provided above
Comment #9
lauriiiCommitted 7f2f2d1 and pushed to 10.1.x. Thanks!