Use `:header-height` prop for top offset in what's new drawer
What does this MR do and why?
Remove custom margin-top
style overrides for What's new drawer, use getContentWrapperHeight
to bind :header-height
on GlDrawer
. !121966 (merged) adds .content-wrapper
as default argument in getContentWrapperHeight
so that implementations of GlDrawer
with :header-height
no longer need to add an argument.
Screenshots or screen recordings
Old Nav | New Nav |
---|---|
How to set up and validate locally
- Enable system header in admin settings
- Toggle the performance bar with p + b
- View "What's new" drawer in help menu
- Confirm top position is correct
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.
Related to #412976 (closed)