Problem/Motivation

  1. 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...
  2. The second issue is - the icon and text are not aligned, see screenshot2
  3. Third issue is - Icon has border-radius in design but in Olivero theme radius is missing

Steps to reproduce

  1. Set up the Drupal site, and moved on Olivero theme.
  2. Switch on a smaller screen (Below 768px)
  3. 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

Issue fork drupal-3382843

Command icon 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

shweta__sharma created an issue. See original summary.

shweta__sharma’s picture

Issue summary: View changes
gauravvvv’s picture

Status: Active » Needs review
StatusFileSize
new64.32 KB
new1.22 KB

I have attached a patch for same, Also attached after patch screenshot for reference. please review

After Patch

smustgrave’s picture

shweta__sharma’s picture

StatusFileSize
new256.96 KB
new1.54 KB
new878 bytes

Applied #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

shweta__sharma’s picture

StatusFileSize
new1.58 KB
new998 bytes

Fixed the #5 error.

smustgrave’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: -Needs subsystem maintainer review

This is actually a small tweak I don't think needs submaintainer review.

Think it's good for committer review.

quietone’s picture

I tested this on Drupal 11.x and didn't notice any difference before and after the patch. Can someone upload a before image?

  • lauriii committed 08f6d675 on 11.x
    Issue #3382843 by shweta__sharma, Gauravvvv, smustgrave: Olivero: RSS...
lauriii’s picture

Version: 10.1.x-dev » 11.x-dev
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed
Issue tags: -rss feed
StatusFileSize
new8.23 KB
new8.07 KB

Added before and after screenshots to issue summary.

Committed 08f6d67 and pushed to 11.x. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.