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

Modal content overflow doesn't fade #2974

Open
eedzjee opened this issue Aug 28, 2024 · 1 comment
Open

Modal content overflow doesn't fade #2974

eedzjee opened this issue Aug 28, 2024 · 1 comment
Labels
component: Modal For Modal issues

Comments

@eedzjee
Copy link

eedzjee commented Aug 28, 2024

Bug: Modal content overflow doesn't fade

Detailed description

When a modal dialog has scrollable content, the content isn't fading out in the Angular variant of the modal.

Is this a feature request (new component, new icon), a bug, or a general issue?

Bug

Is this issue related to a specific component?

Modal (https://angular.carbondesignsystem.com/?path=/story/components-modal--basic)

What did you expect to happen? What happened instead? What would you like to see changed?

A modal dialog with a lot of content should have a fade-out at the bottom, as described in the documentation:
https://carbondesignsystem.com/components/modal/usage/#overflow-content

What browser are you working in?

Chrome / Edge on a Mac

What version of the Carbon Design System are you using?

The current storybook (I guess 5.32.0 at the moment)

What offering/product do you work on? Any pressing ship or release dates we should be aware of?

No pressing dates.

Steps to reproduce the issue

(will create the codesandbox later)

  1. Go to https://angular.carbondesignsystem.com/?path=/story/components-modal--basic
  2. Update the modalText in 'Controls' to add a lot of text
  3. Open the modal
  4. Scroll down the content and see that there is no fade:

Additional information

Screenshot 2024-08-28 at 16 00 14

@eduardmarcinco
Copy link
Contributor

hi, I did a bit of analysis and it seems class .cds--modal-content--overflow-indicator that is used by carbon-components angular was removed from @carbon/styles.

The commit where it was removed: carbon-design-system/carbon@ffc385b#diff-e66e03e601056150086bae060a7b1e96a3e537f83b7f002db1305fcab05b2fbeL359

The class was replaced with .cds--modal-scroll-content that should be applied on content div along with .cds--modal-content.

Carbon react implementation: https://github.com/carbon-design-system/carbon/blob/main/packages/react/src/components/Modal/Modal.tsx#L366-L368

@Akshat55 ^^

@Akshat55 Akshat55 added the component: Modal For Modal issues label Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: Modal For Modal issues
Projects
None yet
Development

No branches or pull requests

3 participants