Skip to content

[FINNA-4092] Check top navbar style definitions#3433

Open
siiriylonen wants to merge 6 commits intoNatLibFi:devfrom
siiriylonen:FINNA-4092-check-navigation-style-definitions
Open

[FINNA-4092] Check top navbar style definitions#3433
siiriylonen wants to merge 6 commits intoNatLibFi:devfrom
siiriylonen:FINNA-4092-check-navigation-style-definitions

Conversation

@siiriylonen
Copy link

navigations.scss and navbar-bootstrap.scss mainly.

@siiriylonen siiriylonen changed the title [FINNA-4092] Check tob navbar style definitions [FINNA-4092] Check top navbar style definitions Feb 5, 2026
@siiriylonen
Copy link
Author

siiriylonen commented Feb 6, 2026

Pääasiallisesti:

  • navigation.scss
  • navbar-bootstrap.scss

Huomioita, muut kuin perus yhdistely/järjestely/poisto:

  • On sekaisin include ja media breakpoint-tapoja, en muutellut jos oli eri.
  • Oli muutama kokonainen luokka joiden en ainakaan huomannut olevan enää missään (esim. .swipe-arrow-navigation, div.logoutOptions (pitäisi olla li), .jump-menu-style ja kaikki joissa oli .open)
  • Olen nyt jaotellut navigation.scss => .finna-navbar ja tietyt menut ja navbar-bootstrap.scss => yleiset navbar säännöt. Jos on joku eri ajatus ollut niille niin voin korjata sen mukaan.
  • Oli aika paljon yhteneväisyyksiä ja päällekäisyyksiä, katsoin että pitäisi toimia ja olla ok, mutta ihan huomiona.

@siiriylonen siiriylonen marked this pull request as ready for review February 6, 2026 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant