Move pin button to be sibling of nav item link in new navigation
What does this MR do and why?
Move pin button to be sibling of nav item link in new navigation
- Move
<button>
to be sibling of<nav-item>
link element - Remove custom
nav-item-link
andnav-item-badge
class usage and styles - Add
show-on-focus-or-hover--control
control class - Add
hide-on-focus-or-hover
context/control/target classes - Update
disable_animations.scss
usage of:not()
selector to removenav-item-link
selector, opt for direct selectorbutton.show-on-focus-or-hover--target
Screenshots or screen recordings
Before | After |
---|---|
CleanShot_2023-09-06_at_15.49.36 | CleanShot_2023-09-06_at_15.48.21 |
CleanShot_2023-09-06_at_15.56.01 | CleanShot_2023-09-06_at_15.58.06 |
How to set up and validate locally
- Review nav item on page with pin-able items e.g. MR or Issue http://gdk.test:3000/flightjs/Flight/-/merge_requests/4
- Validate item with badge still renders pin button on
:hover
/:focus
- Validate item with badge still renders pin button on
- Open "Search or go to..." and validate frequent project/groups list still have remove "-" button display on
:hover
/:focus
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.
Edited by Scott de Jonge