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

Loading layout scroll up #44

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Loading layout scroll up #44

wants to merge 3 commits into from

Conversation

RVany
Copy link
Contributor

@RVany RVany commented Jan 29, 2024

Solved this DevOps Issue

Issue Description
The layout of the loading state move the bottom of the page

We should keep the same space when loading

@RVany RVany linked an issue Jan 29, 2024 that may be closed by this pull request
@RVany RVany requested a review from Steph4104 January 29, 2024 18:11
@RVany RVany changed the title Min Height added to the container Loading layout scroll up Jan 29, 2024
@RVany
Copy link
Contributor Author

RVany commented Feb 20, 2024

image

@RVany
Copy link
Contributor Author

RVany commented Feb 20, 2024

image

@RVany
Copy link
Contributor Author

RVany commented Feb 20, 2024

image
image
image

@RVany
Copy link
Contributor Author

RVany commented Feb 20, 2024

After the discussion, we decided to change the webpart's height according to the number of items per page selected in the setting.

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

Successfully merging this pull request may close these issues.

Loading layout scroll up
1 participant