#573-Fill the screen automatically with rows. #599

Closed
a24danwe wants to merge 0 commits from #573-add-enough-grid-rows into team_2_week_3
Collaborator
No description provided.
Author
Collaborator

The problem was that initially in the edit mode there was not enough rows to fill the screen. This uses the window height and row height to dynamically calculate the minimum number of rows needed, ensuring the grid fills the space.

The problem was that initially in the edit mode there was not enough rows to fill the screen. This uses the window height and row height to dynamically calculate the minimum number of rows needed, ensuring the grid fills the space.
a24danwe changed title from Fill the screen automatically with gridRows. to Fill the screen automatically with rows. 2026-04-27 21:47:45 +00:00
a24danwe changed title from Fill the screen automatically with rows. to #573-Fill the screen automatically with rows. 2026-04-27 21:48:17 +00:00
Collaborator
No description provided.
Collaborator

Review on #573

Whats done

Tested on Windows 11 using Edge

  • Tested edit mode grid behavior
  • App starts correctly
    • Grid fills most of the screen
    • Layout looks stable
    • A small gap is visible above the footer

Inspected following files

  • frontend/src/components/Dashboard/Dashboard.js
    • Indentation/code styling
      • Follows prettier
    • Naming convention
      • Naming is clear and follows convention
    • Comments
      • Comments are understandable
    • General code quality
      • Logic is simple and easy to follow

What needs to be done

  • frontend/src/components/Dashboard/Dashboard.js
    • Grid does not fully fill the screen (gap above footer)
    • The issue is likely caused by the use of a fixed reservedHeight value (250), which may not match the actual height of header/footer on all screens

Conclusion

  • The feature improves the layout
  • However, the grid does not fully fill the screen

Needs minor improvement before merge.

# Review on #573 ## Whats done ### Tested on Windows 11 using Edge + Tested edit mode grid behavior + App starts correctly + Grid fills most of the screen + Layout looks stable + A small gap is visible above the footer ### Inspected following files + frontend/src/components/Dashboard/Dashboard.js + Indentation/code styling + Follows prettier + Naming convention + Naming is clear and follows convention + Comments + Comments are understandable + General code quality + Logic is simple and easy to follow ## What needs to be done + frontend/src/components/Dashboard/Dashboard.js + Grid does not fully fill the screen (gap above footer) + The issue is likely caused by the use of a fixed reservedHeight value (250), which may not match the actual height of header/footer on all screens ## Conclusion + The feature improves the layout + However, the grid does not fully fill the screen Needs minor improvement before merge.
Author
Collaborator

Thank you for the review!

I removed the reservedHeight since I think that the height is not constant.

I adjusted rowheight to include the whitespace between each row, now it is totalRowHeight.

Thank you for the review! I removed the reservedHeight since I think that the height is not constant. I adjusted rowheight to include the whitespace between each row, now it is totalRowHeight.
Collaborator

Tested again after the latest changes.

The calculation is improved, but there is still a visible gap above the footer.

The issue is not fully resolved yet.

Tested again after the latest changes. The calculation is improved, but there is still a visible gap above the footer. The issue is not fully resolved yet.
Collaborator

In order to include this into the next weekly branches I will accept the current changes and create a new issue for this. This wont break anything else.

In order to include this into the next weekly branches I will accept the current changes and create a new issue for this. This wont break anything else.
Collaborator

Merged!

Merged!
a24vinla closed this pull request 2026-04-29 07:07:29 +00:00

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Andras/BoundlessFlowCampus2K!599
No description provided.