Problem/Motivation
- RSS feed block on the smaller screen is not placed as per design. The icon is placed on the right side and the text is placed on the left side in the Olivero theme but in design the placement is different. See in design, attaching design link here - https://www.figma.com/file/x5zBLbvoW1jsvyAOt4Gp9I/Olivero-Theme---Public...
- The second issue is - the icon and text are not aligned, see screenshot2
- Third issue is - Icon has border-radius in design but in Olivero theme radius is missing
Steps to reproduce
- Set up the Drupal site, and moved on Olivero theme.
- Switch on a smaller screen (Below 768px)
- See the RSS feed block at the bottom
Proposed resolution
Need to style as per the design
Remaining tasks
User interface changes
Before:

After:

API changes
Data model changes
Release notes snippet
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | Screenshot 2023-09-03 at 12.09.35 PM.png | 8.07 KB | lauriii |
| #10 | Screenshot 2023-09-03 at 12.08.35 PM.png | 8.23 KB | lauriii |
| #6 | interdiff_5-6.txt | 998 bytes | shweta__sharma |
| #6 | 3382843-6.patch | 1.58 KB | shweta__sharma |
| #5 | interdiff_3-5.txt | 878 bytes | shweta__sharma |
Issue fork drupal-3382843
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
shweta__sharma commentedComment #3
gauravvvv commentedI have attached a patch for same, Also attached after patch screenshot for reference. please review
After Patch
Comment #4
smustgrave commentedComment #5
shweta__sharma commentedApplied #3 patch and it is working fine, the feed block is now as per design. Added a screenshot for reference, also I provided the patch for the third point which is related to border-radius, please review.
Thanks
Comment #6
shweta__sharma commentedFixed the #5 error.
Comment #7
smustgrave commentedThis is actually a small tweak I don't think needs submaintainer review.
Think it's good for committer review.
Comment #8
quietone commentedI tested this on Drupal 11.x and didn't notice any difference before and after the patch. Can someone upload a before image?
Comment #10
lauriiiAdded before and after screenshots to issue summary.
Committed 08f6d67 and pushed to 11.x. Thanks!