KohaSuomi / Koha-22x

(K22) Versioon 22.11 siirtyminen. Wiki-osiossa on erilaisia ohjeistuksia ja tietoa muutoksista ja uusista ominaisuuksista.
GNU General Public License v3.0
0 stars 0 forks source link

Sivutus tietueen Varaukset-sivulle - ksdev/ks-0015-bug-23269 #56

Closed emta001 closed 1 year ago

emta001 commented 1 year ago

Siirrettävä ominaisuus

Kohan oma Varaukset-sivu on liian hidas ladata. Koha-Suomen versio toimii APIn kautta jotta lista voidaan sivuttaa.

emta001 commented 1 year ago

Tuotu testeille.

ajrajala commented 1 year ago

Toisen tiketin testauksen yhteydessä huomasin, että suomenkielellä Kohaa käytettäessä varausjono ei lataudu laisinkaan eikä siihen auta vaikka sivun lataisi tai tietueen hakisi uudestaan. "Näytä aina varaukset" -rastin klikkailu ei myöskään auta. Muistelen aiemmin varausten latautuneen suomeksikin käyttäessä, joten voiko olla että sivutus on rikkonut tässä jotain?

emta001 commented 1 year ago

Toisen tiketin testauksen yhteydessä huomasin, että suomenkielellä Kohaa käytettäessä varausjono ei lataudu laisinkaan eikä siihen auta vaikka sivun lataisi tai tietueen hakisi uudestaan. "Näytä aina varaukset" -rastin klikkailu ei myöskään auta. Muistelen aiemmin varausten latautuneen suomeksikin käyttäessä, joten voiko olla että sivutus on rikkonut tässä jotain?

Siellä ei ole vielä meidän käännökset paikallaan, voi johtua siitä. Ajatuksena on ollut siirtää käännökset omaan repoon, mutta toteutus on vielä hieman vaiheessa.

ajrajala commented 1 year ago

Sellainen huomio, että Varaukset-sivulla on nyt ainoastaan mahdollista siirtyä seuraavalle/edelliselle sivulle, mutta ei suoraan tietylle sivulle (tuotannossa olevat sivunumerolinkit siis puuttuu).

emta001 commented 1 year ago

Sellainen huomio, että Varaukset-sivulla on nyt ainoastaan mahdollista siirtyä seuraavalle/edelliselle sivulle, mutta ei suoraan tietylle sivulle (tuotannossa olevat sivunumerolinkit siis puuttuu).

Tutkin tätä joku aika sitten ja vaikuttaisi sille, että käyttöliittymämuutoksen myötä kaikki Kohan taulukot käyttäytyvät nyt näin. En saanut myöskään numerointia lisättyä erikseen tuonne Varaukset-sivulle. Hieman köpsä muutos, pitää vielä tutkia miksi ihmeessä tämmöinen muutos on tuonne viety.

AnneliO commented 1 year ago

Hmm.. Vaaran nextillä näkyvät kyllä sivunumerot ja niitä voi myös käyttää.

kuva

emta001 commented 1 year ago

Hmm.. Vaaran nextillä näkyvät kyllä sivunumerot ja niitä voi myös käyttää.

kuva

Outoa, mikähän niihin sitten vaikuttaa. Mitenhän muilla nexteillä mahtaa olla?

AnneliO commented 1 year ago

OUTIssa ei näy: kuva

emta001 commented 1 year ago

OUTIssa ei näy: kuva

Eli Vaarassa on nyt jotain eri tavalla. Sen kun osais taas arvata että mikä :D

Kati-Helle commented 1 year ago

Hellessäkään ei näy sivunumerolinkkejä kuva

AnneliO commented 1 year ago

Tyhjensin Evästeet ja sivustotiedot sekä Historiatiedot Firefoxin asetuksista, niin sivunumerot tulivat mulla näkyviin OUTIssa, missä ne ei ennen tyhjennystä näkyneet.

Kokeiletteko muutkin? :)

Kati-Helle commented 1 year ago

Minulle sivunumerot eivät edelleenkään näy.

AnneliO commented 1 year ago

Minulle sivunumerot eivät edelleenkään näy.

Varmistan, että poistithan kaikki historiatiedot, ei vain osaa? :)

Ja selaimen kiinni tyhjennyksen jälkeen?

Kati-Helle commented 1 year ago

Kyllä vain ja kirjauduin noin tehden ulos kaikista toiminnoita, joihin olin kirjautunut :) Tein nyt vielä nuo kaikki poistamiset uudestaan ja suljin selaimen.

Mutta ei edelleenkään sivunumeroita näkyvillä kuva

ajrajala commented 1 year ago

Minulla sivunumerot on palanneet takaisin näkyville. Evästeet ja sivustotiedot sekä historiatiedot on tässä välissä tyhjentyneet automaattisesti, joten hyvinkin saattoi olla niistä kiinni.

emta001 commented 1 year ago

Minulla sivunumerot on palanneet takaisin näkyville. Evästeet ja sivustotiedot sekä historiatiedot on tässä välissä tyhjentyneet automaattisesti, joten hyvinkin saattoi olla niistä kiinni.

Näkyykö siulla myös "First" ja "Last" namiskat? Nekin näemmä puuttuvat.

ajrajala commented 1 year ago

Nekin painikkeet näkyvät (ja toimivat). kuva

AnneliO commented 1 year ago

Nekin painikkeet näkyvät (ja toimivat). kuva

Minusta tuossa kuvassa ei ole Viimeinen ja Ensimmäinen -kohtia, ellen ihan sokia ole. ;)

ajrajala commented 1 year ago

Äh, sori, ajattelin koko ajan noita edellinen ja seuraava -painikkeita. Meillä näkyy tuotannossakin pelkästään ne?

emta001 commented 1 year ago

Hmm, luulis että nekin ois tullut ton tyhjentelyn jälkeen näkyville jos ovat näkyäkseen. Siellä on varmaan vielä jotain mikä kinnaa.

velimarj commented 1 year ago

OUTIssa myös numerot ja edellinen/seuraava -painikkeet näkyvät ja toimivat. Noita ensimmäinen/viimeinen -painikkeita puolestaan ei näy meilläkään nextillä tai tuotannossa.

emta001 commented 1 year ago

Jaahas :D no sitten ne eivät oo olleet siellä käytössä syystä tai toisesta. Tarvitsisiko olla?

AnneliO commented 1 year ago

Sanoisin, että koska olemme pärjänneet ilman tähänkin mennessä, niin tutkitaan ongelmaa versionvaihdon jälkeen. :)

emta001 commented 1 year ago

Hmmm, laitetaanko tälle tiketille sulku vai odottaa tila?

AnneliO commented 1 year ago

Jos tekis noista first/last-namiskoista oman tikettinsä ja jos kukaan ei keksi mitään ongelmia liittyen sivutukseen, niin sitten vois sulkea tän tiketin.

emta001 commented 1 year ago

Jos tekis noista first/last-namiskoista oman tikettinsä ja jos kukaan ei keksi mitään ongelmia liittyen sivutukseen, niin sitten vois sulkea tän tiketin.

Tehdään näin.

AnneliO commented 1 year ago

Sellainen tuli mieleen, että se taustatyö-ilmoitus ei näy edelleenkään. Oliko meillä tarkoitus saada se näkyville?

emta001 commented 1 year ago

Sellainen tuli mieleen, että se taustatyö-ilmoitus ei näy edelleenkään. Oliko meillä tarkoitus saada se näkyville?

Muistelisin, että siitä on tiketti ihan täällä versionvaihdossa.

MikkoLiimatainen commented 1 year ago

Sivunumerointi tulee näkyviin tiedostossa https://vaski-next.koha-suomi.fi/intranet-tmpl/prog/css/staff-global_22.1100001.css olevan säännön takia:

@media only screen and (min-width:1500px) {
 .dataTables_wrapper .dataTables_paginate span .ellipsis,
 .dataTables_wrapper .dataTables_paginate span .paginate_button {
  display:inline-block
 }
}

Samassa tiedostossa näyttää olevan sääntö, joka piilottaa sivunumerot:

.dataTables_wrapper .dataTables_paginate span .ellipsis,
.dataTables_wrapper .dataTables_paginate span .paginate_button {
 display:none
}

Eli sivun leveyden tulee olla vähintään 1500px, jotta sivunumerointi näkyy. Voisikohan tuota 1500px pudottaa hieman pienemmäksi, jotta sivunumerointi toimisi pienemmissäkin ikkunoissa?

emta001 commented 1 year ago

Eli sivun leveyden tulee olla vähintään 1500px, jotta sivunumerointi näkyy. Voisikohan tuota 1500px pudottaa hieman pienemmäksi, jotta sivunumerointi toimisi pienemmissäkin ikkunoissa?

Hmm, pitää tutkia. En oo ihan varma kuinka näihin tehdään muutoksia (ja tätä pitäis myös ehdottaa yhteisöön).

lasse-koha commented 1 year ago

Sellainen tuli mieleen, että se taustatyö-ilmoitus ei näy edelleenkään. Oliko meillä tarkoitus saada se näkyville?

Taustatyöilmoituksen näkymättömyyden syy on edelleen arvoitus: https://github.com/KohaSuomi/Koha-22x/issues/37

emta001 commented 1 year ago

Sivunumerointi tulee näkyviin tiedostossa https://vaski-next.koha-suomi.fi/intranet-tmpl/prog/css/staff-global_22.1100001.css olevan säännön takia:

@media only screen and (min-width:1500px) {
 .dataTables_wrapper .dataTables_paginate span .ellipsis,
 .dataTables_wrapper .dataTables_paginate span .paginate_button {
  display:inline-block
 }
}

Samassa tiedostossa näyttää olevan sääntö, joka piilottaa sivunumerot:

.dataTables_wrapper .dataTables_paginate span .ellipsis,
.dataTables_wrapper .dataTables_paginate span .paginate_button {
 display:none
}

Eli sivun leveyden tulee olla vähintään 1500px, jotta sivunumerointi näkyy. Voisikohan tuota 1500px pudottaa hieman pienemmäksi, jotta sivunumerointi toimisi pienemmissäkin ikkunoissa?

Tämä pitää tosiaan korjata yhteisössä asti, mutta sen aikaa seuraavan rimpsun pitäisi korjata tilanne:

@media only screen and (min-width: 1000px) {
    .dataTables_wrapper
        .dataTables_paginate
            span
                .paginate_button,
                .ellipsis {
                    display: inline-block;
                }
}

Tuota min-widthiä voi säätää tarpeen mukaan.

AnneliO commented 1 year ago

Lisäsin Emmin rimpsun IntranetUserCSS-wikiin vahvasti suositeltaviin.

MikkoLiimatainen commented 1 year ago

Nähdäkseni tuon rimpsun tulee olla tässä muodossa, jotta tuo ellipsis toimii:

@media only screen and (min-width: 1000px) {
    .dataTables_wrapper
        .dataTables_paginate
            span
                .paginate_button,
    .dataTables_wrapper
        .dataTables_paginate
            span
                .ellipsis {
                    display: inline-block;
                }
}

Testasin tuota nidehaun tuloksissa, joka myös käyttää näitä paginate-linkkejä. Suurella tulossivujen määrällä nuo kolme pistettä (ellipses) katosivat alkuperäisellä rimpsulla eri tahtiin sivun kokoa muutettaessa. Tämä rimpsu korjaa tuon ongelman.

AnneliO commented 1 year ago

Vaihdoin IntranetUserCSS-sivulle Mikon päivitetyn version.

MikkoLiimatainen commented 1 year ago

Näyttäisi siltä, että ainakin tietyillä tablettimallella tuo min-width täytyy pudottaa jonnekin 600px tuntumaan, jotta saa pystyasennossa sivunumeroinnin näkymään esim. hyllyvaraukset-raportilla. Mutta tuo on kyllä hyvä, että sen voi omien tarpeiden mukaan säätää. Mihinkään poikkeukselliseen toimintaan en toistaiseksi tuon testailussa ole törmännyt, joten min-width -arvon voinee melko huoletta valita pienimmän laitteen resoluution mukaan.

AnneliO commented 1 year ago

Onko tässä vielä jotain vai voiko tiketin sulkea?

emta001 commented 1 year ago

Onko tässä vielä jotain vai voiko tiketin sulkea?

Hmm, ainakin tiketissä mainitut ongelmat on suurimmilta osin korjattu. Toki sivulistalta uupuu vielä "First" ja "Last" namiskat, mutta voisiko siitä avata oman tiketin (jos on tarvetta)?

AnneliO commented 1 year ago

Tein first/last-ongelmasta tiketin Koha-repositorioon.

Suljen tämän tiketin, avataan tarvittaessa uusia.