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

KModal does not shrink when the content has a smaller height #570

Closed
AlexVelezLl opened this issue Mar 6, 2024 · 5 comments
Closed

KModal does not shrink when the content has a smaller height #570

AlexVelezLl opened this issue Mar 6, 2024 · 5 comments
Assignees
Labels
Component: KModal P2 - normal Priority: Nice to have

Comments

@AlexVelezLl
Copy link
Member

Product

Studio

Expected behavior

The height of the KModal should dynamically adjust to the height of the content it renders.

Actual behavior

If for some reason the content of the KModal is expanded after its first render, the KModal is expanded accordingly, but if it is then shrinked, the KModal remains with the height of the expanded content, and this can lead to cases like this:

If initially we have a componente with a description like this:

image

But then we want to delete it, we get this view:

image

Cases like this occur in several places where the height of some element varies while the modal is displayed.

Steps to reproduce the issue

  1. Go to Studio and checkout the branch studio-usability-enhancements
  2. Edit the content nodes of a channel
  3. Edit the title and description of a node

Environment

  • OS: Ubuntu
  • Browser version: Chrome Version 122.0.6261.94 (Official Build) (64-bit)
@AlexVelezLl AlexVelezLl added P2 - normal Priority: Nice to have Component: KModal labels Mar 7, 2024
@shrudge
Copy link

shrudge commented Mar 14, 2024

hey @AlexVelezLl great notice, but I think you listed this issue in the wrong Repo, correct me if am i wrong tho, I am really new to open source so..

@AlexVelezLl
Copy link
Member Author

Hi @shrudge! No, the issue is in the correct repository. KDS is responsible for providing the components that will be used in other Kolibri products, you can learn more about KDS in the repo readme.

The issue is about the KModal component, here in KDS, probably it has something to do with the updateContentSectionStyle method.

@galovdev
Copy link
Contributor

@AlexVelezLl Can I get this issue assigned?

@AlexVelezLl
Copy link
Member Author

Hi @galovdev! Sure! Let me know if you have any question :)

@MisRob
Copy link
Member

MisRob commented May 31, 2024

Closed by #590

@MisRob MisRob closed this as completed May 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: KModal P2 - normal Priority: Nice to have
Projects
None yet
Development

No branches or pull requests

4 participants