Skip to content

[FINNA-4092] Check navigation style definitions#3516

Merged
EreMaijala merged 25 commits into
NatLibFi:devfrom
siiriylonen:FINNA-4092-check-navigation-style-definitions
Jun 1, 2026
Merged

[FINNA-4092] Check navigation style definitions#3516
EreMaijala merged 25 commits into
NatLibFi:devfrom
siiriylonen:FINNA-4092-check-navigation-style-definitions

Conversation

@siiriylonen
Copy link
Copy Markdown

@siiriylonen siiriylonen commented May 26, 2026

finna-navbar.scss, mobile-toolbar.scss, page-selectors.scss, tabs.scss & navs-bootstrap.scss

@siiriylonen
Copy link
Copy Markdown
Author

siiriylonen commented May 28, 2026

page-selectors.scss tuolla näyttää, että puuttuu tyhjä rivi lopusta, vaikka näin ei ole?

Tehtyä:

  • Vaihdoin sinne tab kuitenkin sen > li => > .nav-item ja lisäsin sen luokan sinne login-ikkunaankin, koska selkeämpi.
  • Poistettu pagination-background liittyvät luokka ja muuttuja ...pitäisikö ne (se aiempi grid-breakpoint) tässä poistaa myös tuolta customin variables.scss?

Muuta:

  • .finna-main-tabs oli muutama tietty .active tooltip-nappi laitettu eri värillä?
  • .details-tab ei näytä olevan olemassa, mutta näkyy olevan yhden testin valitsimessa mukana?
  • .nav-tabs on <i>, mutta en ihan saanut selvää mitä siihen tulisi? Sille on oma yksittäinen mobiilimäärittely.
  • Poistin .tabs-responsive, mainitsen koska sitä löytyi kahdesta näkymästä, mutta en itse sivuilta löytänyt.

@siiriylonen siiriylonen marked this pull request as ready for review May 28, 2026 10:20
@EreMaijala EreMaijala merged commit fb6401d into NatLibFi:dev Jun 1, 2026
7 checks passed
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.

3 participants