fix(frontend): keep admin page height consistent#6035
Conversation
|
👋 Thanks for your first contribution to Texera, @Amer-Mukhtar! If you're looking for a good place to start, browse issues labeled You can drive common housekeeping yourself by commenting one of these commands on its own line:
Each command must match exactly: |
Automated Reviewer SuggestionsBased on the
|
|
Hi @kz930, please review this when you have time. It’s a quick fix to stop the admin page layouts from shifting. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6035 +/- ##
============================================
- Coverage 56.82% 56.82% -0.01%
Complexity 3023 3023
============================================
Files 1126 1126
Lines 43708 43708
Branches 4733 4733
============================================
- Hits 24837 24835 -2
Misses 17402 17402
- Partials 1469 1471 +2
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
/request-review @kz930 |
What changes were proposed in this PR?
The admin pages didn't all have the same height, so the layout shifted when switching between sections. This meant the buttons could move around and you'd have to move your cursor to click them.
The proposed change makes the admin pages keep a consistent height, so the layout stays put when moving between sections. It's a small CSS change to the shared dashboard content area.
The fix makes nz-content a bounded flex scroll container, so overflowing content scrolls inside the content area while the sidebar and top nav stay fixed and the page height stays consistent across all admin pages. The nz-content rule now adds min-height: 0 and flex: 1 alongside the existing overflow-y: auto.
Any related issues, documentation, discussions?
Closes #3588
How was this PR tested?
Tested manually
Was this PR authored or co-authored using generative AI tooling?
Nope.