Skip to content

[FINNA-4746] Add scrollable link list component - #3529

Open
siiriylonen wants to merge 8 commits into
NatLibFi:devfrom
siiriylonen:finna-4746-add-new-browse-bar
Open

[FINNA-4746] Add scrollable link list component#3529
siiriylonen wants to merge 8 commits into
NatLibFi:devfrom
siiriylonen:finna-4746-add-new-browse-bar

Conversation

@siiriylonen

Copy link
Copy Markdown

No description provided.

@siiriylonen

Copy link
Copy Markdown
Author

Katsoin tähän sitä nykyistä browsebaria ja mitä muissa komponenteissa voi yleisesti muokata. Eli käyttäjä voi halutessaan lisätä otsikon sekä määritellä otsikkotason ja ylimmän elementin luokan. Linkit voi lisätä linkkinä tai dropdown-valikkona, jossa lisää linkkejä. Onko riittävä/tarvitseeko kaikkia?

On myös ikoni mahdollisuus, mutta tuossa selaa-palkissa käytetty enemmänkin niitä iconlabel:before ikoneita, joten en tiedä halutaanko tähän mielummin ne?
Oliko myös jonkinlainen oletus ulkoasu? koska tällä hetkellä vain tekstilinkkinä ja voi muuttaa tarkemmin muutaman asetetun luokan avulla.

(Ajatuksena myös oli, että tästä pr tulisi siihen FINNA-4741 nuo js ja tyylimuokkaukset.)

@siiriylonen
siiriylonen marked this pull request as ready for review July 15, 2026 07:16
@EreMaijala

Copy link
Copy Markdown

Templaatin sisennyksen voisi laittaa kahdella välilyönnillä, se on yleisemmin käytetty.

Varmaan jos tällä korvataan nykyinen etusivun browsebar, niin olisi hyvä tehdä lisäksi tuki sen konffaukselle samantyyppisellä konfiguraatiotiedostolla kuin ylävalikon HeaderBar.yaml ja footerin FooterMenu.yaml.

@pasitiis

Copy link
Copy Markdown

Tehdään täsä geneerisempi kuin nykyinen selaa palkki. Monissa näkymissä selaa palkkia on käytetty ns. lisävalikkona jolloin se on toiminut vähän kehnosti.
Otsikkotaso, ylimmän elementin luokka sekä linkit linkkeinä tai dropdown-valikkona on mielestäni riittävä taso.

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