Navigation Menu

Skip to content

Commit

Permalink
Make sure that we dont trap users on search.nixos.org
Browse files Browse the repository at this point in the history
and that there is a nice way back to main website
also the navigation is now the same hight as on nixos.org

fixes #59
  • Loading branch information
garbas committed Jun 11, 2020
1 parent 7f2ed39 commit c5cd245
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 17 deletions.
22 changes: 12 additions & 10 deletions src/Main.elm
Expand Up @@ -238,10 +238,13 @@ view model =
[ div [ class "navbar navbar-static-top" ]
[ div [ class "navbar-inner" ]
[ div [ class "container" ]
[ a [ class "brand", href "https://search.nixos.org" ]
[ a [ class "brand", href "https://nixos.org" ]
[ img [ src "https://nixos.org/logo/nix-wiki.png", class "logo" ] []
]
, viewNavigation model.page model.url
, div [ class "nav-collapse collapse" ]
[ ul [ class "nav pull-left" ]
(viewNavigation model.page model.url)
]
]
]
]
Expand All @@ -263,7 +266,7 @@ view model =
]


viewNavigation : Page -> Url.Url -> Html Msg
viewNavigation : Page -> Url.Url -> List (Html Msg)
viewNavigation page url =
let
preserveSearchOptions =
Expand All @@ -288,13 +291,12 @@ viewNavigation page url =
|> List.append preserveSearchOptions
|> Url.Builder.absolute [ path ]
in
ul [ class "nav" ]
(List.map
(viewNavigationItem url)
[ ( createUrl "packages", "Packages" )
, ( createUrl "options", "Options" )
]
)
List.map
(viewNavigationItem url)
[ ( "https://nixos.org", "Back to nixos.org" )
, ( createUrl "packages", "Packages" )
, ( createUrl "options", "Options" )
]


viewNavigationItem :
Expand Down
16 changes: 9 additions & 7 deletions src/index.scss
Expand Up @@ -14,16 +14,18 @@ footer {
height: 4rem;
}

header .navbar {
a.brand {
line-height: 1.5em;
header .navbar.navbar-static-top {
.brand {
padding-bottom: 0;
}
img.logo {
height: 1.5em;
margin-right: 0.5em;
margin-top: -5px;
padding-right: 5px;
line-height: 25px;
height: 25px;
}
ul.nav > li > a {
line-height: 2.5em;
ul.nav > li {
line-height: 20px;
}
}

Expand Down

0 comments on commit c5cd245

Please sign in to comment.