Olivero: The styling of the pager is not matching the design.

Created on 13 September 2023, over 1 year ago
Updated 16 October 2023, about 1 year ago

Problem/Motivation

I was reviewing the design and I observed that the styling of the pager is different from the design. In the design, there is a border defined on the items but in the current implementation, the styling is different. See the attached screenshot for reference.

Steps to reproduce

  1. Set up the Drupal site, and moved on Olivero theme.
  2. Add some content (Create articles and a basic page)
  3. See the pager styling (match with the design)

Proposed resolution

Need to style as per the design

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

📌 Task
Status

Needs work

Version

11.0 🔥

Component
Olivero 

Last updated about 16 hours ago

Created by

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @shweta__sharma
  • Status changed to Needs review over 1 year ago
  • last update over 1 year ago
    Custom Commands Failed
  • I have attached a patch for the same, also attached after patch screenshot for reference. please review

  • last update over 1 year ago
    Custom Commands Failed
  • Status changed to Needs work over 1 year ago
  • 🇺🇸United States smustgrave

    I wouldn’t recommend digging through the figmas so much. There were design changes made before launch that aren’t included. So there’s a chance these can be closed won’t fix.

  • 🇮🇳India hamid.ali

    I have tested the patch mentioned in Comment 3 and have identified several discrepancies when compared to the Figma design:
    - The total number of visible items on Figma is 7
    - The selected item colour, is not the same as per the Figma
    - On hover, there is a blue border on the Figma which is missing
    - When the first item is selected prev and the first buttons are hidden not disabled like in the Figma
    - There are no ellipses on the Figma which is present in this case
    - The colour of the numbers of the pagination item on select and hover are black in this case colour does not match as well

  • Status changed to Needs review about 1 year ago
  • 🇮🇳India Vinayak.Ambig

    smustgrave with respect #4 can you please provide the reference where we may verify the deviations. According to that we can progress.

  • Status changed to Needs work about 1 year ago
  • 🇺🇸United States smustgrave

    They aren’t in the figmas anywhere but talking with the maintainer I know some changes were made. Which is why I wouldn’t recommended digging through the figmas looking for differences. 9/10 are probably working as designed.

    So unless this provides an improvement, which haven’t tested, not sure this will make it. But could be wrong

Production build 0.71.5 2024