Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MenuSegmentTitle Dose not render the correct color for full screen view #6789

Open
1 task done
elvis-iacobescu-sage opened this issue Jun 19, 2024 · 2 comments
Open
1 task done

Comments

@elvis-iacobescu-sage
Copy link

elvis-iacobescu-sage commented Jun 19, 2024

Description

We are rendering dynamically a ScrollableBlock or a menuSegmentTitle. Both have variant="alternate" as a property.
the MenuItem rendered are rendered as children and have variant="alternate" as a property.

The bug reproduces on small screens when the menu is in full screen.

Expected: the background colors for the menu items in menuSegmentTitle to be gray when the menu is in full-screen View.

Current: only the title in menuSegmentTitle is gray, and the rest is black.

Reproduction

https://stackblitz.com/edit/parsium-carbon-starter-viwky9?file=src%2Fmain.tsx

Steps to reproduce

To reproduce:
open menu. (click the test button)

You will notice that the MenuSegmentTitle items are not gray .
if you click the menu item on top of the menuSementTitle you will switch to the scrollable block where the menu items are displayed properly

JIRA ticket numbers (Sage only)

sbs-36740

Suggested solution

No response

Carbon version

138.2.0

Design tokens version

No response

Relevant browsers

Chrome

Relevant OSs

Windows

Additional context

In Stackblits seems that a padding is added for some reason on all li elements. it is not happening in carbon documentation or our application

Confidentiality

  • I confirm there is no confidential or commercially sensitive information included.
@elvis-iacobescu-sage elvis-iacobescu-sage added Bug triage Triage Required labels Jun 19, 2024
@nineteen88
Copy link
Contributor

FE-6679

@nineteen88 nineteen88 added On Backlog Squad Nebula and removed triage Triage Required labels Jun 25, 2024
@edleeks87
Copy link
Contributor

FYI we're just waiting for a steer from UX/DS on this once we have that the ticket can be refined and pulled into sprint

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants