Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FINNA-2472] & [FINNA-2603] Alter/Fix collection list tab search bar #3044

Open
wants to merge 11 commits into
base: dev
Choose a base branch
from

Conversation

siiriylonen
Copy link

No description provided.

@siiriylonen
Copy link
Author

Record-templaatissa vähemmän tilaa, mutta yritin tässä asetella niitä vähän paremmin siihen. (Jos pitäisi jotenkin suuremmin muutella niin ehkä erotan sen 2603 tästä..)

Tuolla mobile-toolbar.less noihin negatiivisiin marginaaleihin tuli arvoja katsoessa joku "Invalid property value" ja katsoin että se on myös dev, mutta ei tuotannossa -> siksi muutettu sitä.

@siiriylonen siiriylonen marked this pull request as ready for review October 1, 2024 11:36
@pasitiis
Copy link

pasitiis commented Oct 8, 2024

Jäin pohtimaan, onko tarpeellista pudottaa hakuikoni pois mobiilissa kun käytännössä rivittyy aina kahdelle riville, ja ikonia varten jäisi tilaa.
Näyttökuva 2024-10-08 kello 13 06 16

Huomasin myös että Kokoelman sisällön järjestä valinnat ei toimi (sama myös tuotannossa tällä hetkellä).

Muutoksia on aika paljon (verratuna yläjuoksun toteutukseen), pohdin että aiheuttaako muutokset lisätyötä tulevaisuudessa.

@siiriylonen
Copy link
Author

Järjestä-nappiin liittyen tein jo tiketin, FINNA-2652. (Se järjestämisen osoite lähtee kokonaan pois /CollectionList.)

Copy link

@pasitiis pasitiis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tarkastelin pr:ää vain pintapuolisesti. Sivut näytti nyt skaalautuvan paremmin.

Copy link

@EreMaijala EreMaijala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Muuten ok, mutta miksi pelkästään collectionlist:iä varten tehdään erilaiset kontrollit view:lle ja sorttaukselle? Olisi parempi pyrkiä yhtenäisyyteen eikä tehdä sivukohtaisesti eri tavalla.

?>
<?php if (count($viewList) > 1): ?>
<div class="view-option-container text-right">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Täällä on jossain kohdassa jotain pielessä, koska sisennykset ja lopussa oleva endif eivät sovi yhteen.

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