Problem/Motivation
Block layout overflow issue and Incorrect style for all draggable table on mobile.
Steps to reproduce
1. login as admin
2. use claro or olivero (issue in both)
3. goto => structure => block layout
4. switch to mobile device you will see contents are overflowing outside page from right.
Proposed resolution
the fix is to add overflow scroll to table wrapper
Remaining tasks
Need review and test
User interface changes
scroll added for table and align for text and buttons are also fixed
Release notes snippet
Drupal version 11
| Comment | File | Size | Author |
|---|---|---|---|
| #36 | after-mr-360-width.jpg | 88.99 KB | shivam_tiwari |
| #36 | before-mr-360-width.jpg | 98.29 KB | shivam_tiwari |
| #36 | after-mr-280-width.jpg | 88.55 KB | shivam_tiwari |
| #36 | before-mr-280-width.jpg | 89.65 KB | shivam_tiwari |
| #35 | Screencast from 2025-03-24 11-25-39.gif | 753.58 KB | sandip |
Issue fork drupal-3193332
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:
Comments
Comment #2
djsagar commentedComment #3
djsagar commentedI create patch to relative this issue please review and give feedback.
Thanks!
Comment #4
abhijith s commentedApplied patch #3.Adding screenshots below.
Before patch:
After patch:
In the last image you can see that the issue is still there.It works well in other cases.Still needs work.
Comment #5
abhijith s commentedComment #6
djsagar commentedWell for 320px of mobile i upload video after applied patch and not able to replicate 320px mobile issue please watch video.
Still you found the issue try Caches cleared then check.
Thanks!
.
Comment #11
smustgrave commentedThis issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request as a guide.
Think this will need to be updated for D10 themes as bartik and seven have been removed.
Tagging for tests to show the issue.
Believe this still could be issue.
Comment #13
akashdab commentedThis issue still persists in both Claro and Olivero themes.
While in both of them the layout content overflows, in my testing it is much worse with olivero; it makes the overflowing content illegible with the layout content not taking on white background color like it does in claro.
Link to test with Claro.
Link to test with Olivero.
Comment #16
saurav-drupal-dev commentedI have created MR with the fix for both themes please review.
Before


After


Comment #17
smustgrave commentedComment #18
saurav-drupal-dev commentedComment #19
saurav-drupal-dev commentedComment #20
saurav-drupal-dev commentedComment #21
smustgrave commentedUI changes should be included in an issue summary. Adding a sentence doesn't count. Before/after screenshots should be added
Comment #22
djsagar commentedI have checked MR #15 in Drupal 11.
It fixed the Olivero issue on mobile, but the issue still persists in Claro.
Olivero

Before MR
After MR

Claro

Before MR
After MR

Comment #23
djsagar commentedRemoved the tag and added a screenshot, but the issue still persists.
Comment #24
saurav-drupal-dev commentedi have fixed the issue mentioned @djsagar
before

after

Comment #27
saurav-drupal-dev commentedComment #28
smustgrave commentedComment #29
asawari commentedTest status - Pass
Verified an issue on Drupal 11.X version for Olivero and claro themes. MR applied successfully. Issue looks fixed.
Followed steps below-
1. login as admin
2. use claro or olivero
3. goto => structure => block layout
4. switch to mobile device you will see contents are overflowing outside page from right.
Attaching screenshots for reference.
RTBC+1
Comment #30
asawari commentedComment #31
saschaeggiWe have scrollable tables with scroll indicators in Gin
So we might want to consider re-using Gin’s implementation here also to avoid potential conflicts and issues
Comment #32
smustgrave commentedShould investigate #31since it came from a claro maintainer
Comment #33
sandip commentedI am working here to implement Gin's process as mentioned in #31.
Comment #34
sandip commentedAnother thing i want to ask that changes should be done for only Claro theme right? it does not make any sense to implement the same for Olivero as it is not a admin theme. Please correct me if i am wrong.
Comment #35
sandip commentedHi @saschaeggi, @smustgrave,
I have worked on fixing the issue and add some change to
table.html.twigfile as i applied a parent div to the table so i can applyoverflow-x: autoto it as same as Gin. This issue is not only for Block layout page this issue will occur everywhere where table twig file is rendering so i had to change the table.html.twig file. As i see i also need to change theview-view-table.html.twigalso as the same issue is coming from it. So should i change the views table twig file in this issue queue?In the current MR, I haven't applied the changes for the
Oliverotheme. Please let me know if you would like me to include those adjustments as well.Thank you!
Comment #36
shivam_tiwari commentedI checked this issue and MR. Faced same problem on Drupal fresh set up. After applying MR as a patch it resolves problem. Moving this issue to RTBC as it looks fine.
Adding some screenshots related to my testing.
Comment #37
shivam_tiwari commentedOhh, Current MR is not applied for Olivero theme. Moving this to Needs Review.
@saschaeggi, @smustgrave, Please check and provide your feedback.
Comment #38
sandip commentedhttps://www.drupal.org/project/drupal/issues/3068696 I found this issue which is already working for fixing the table responsive issue for claro theme. I think we can close or Duplicate this issue. @smustgrave, can you please look into it.
Comment #39
smustgrave commentedAgree that issue seems like this was a duplicate.