Skip to content

Commit

Permalink
fix sidenav compact group margins
Browse files Browse the repository at this point in the history
  • Loading branch information
charbelrami committed Jan 12, 2024
1 parent b6cb324 commit 97bd8bc
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions src/Nri/Ui/SideNav/V5.elm
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,7 @@ import Accessibility.Styled.Aria as Aria
import Accessibility.Styled.Style as Style
import ClickableAttributes exposing (ClickableAttributes)
import Css exposing (..)
import Css.Global
import Css.Media
import Html.Styled
import Html.Styled.Attributes as Attributes
Expand Down Expand Up @@ -517,7 +518,12 @@ viewSidebarEntry config extraStyles entry_ =
, viewRightIcon groupConfig
]
, ul
[ Attributes.css [ margin zero, padding zero, listStyle none ]
[ Attributes.css
[ margin zero
, padding zero
, listStyle none
, Css.Global.children [ Css.Global.typeSelector "li" [ margin2 (px 4) zero ] ]
]
, Aria.describedBy [ groupTitleId ]
]
(List.map
Expand All @@ -526,8 +532,6 @@ viewSidebarEntry config extraStyles entry_ =
:: fontWeight (int 400)
:: extraStyles
)
-- >> List.singleton
-- >> span [ Attributes.css [ listStyle none, margin2 (px 4) zero ] ]
)
children
)
Expand Down

0 comments on commit 97bd8bc

Please sign in to comment.