[FINNA-774] Make advanced search facets collapsible #2637
Conversation
|
Täällä oli pienoinen ongelma liittyen Vuosivalinta- ja Kartta-fasettiin. Kun ne olivat alussa collapsed tilassa, niin niiden asetuksiin liittyviä margin arvoja yms. ei laskettu oikein (aina 0), jonka takia ne näkyivät väärin. En osaa sanoa kumpi on parempi ratkaisu, että ne vain olisivat aluksi auki (koska sen jälkeen ei ongelmaa sulkemisen ja avaamisen kanssa) vai kuin ovat nyt (esim. mitä jos fasetin lataamisessa kestää). |
There was a problem hiding this comment.
Ohessa pari pientä kommenttia nykyisestä toteutuksesta, mutta tässä pari isompaa juttua:
- Olisi parempi käyttää finna-panel -komponenttia eikä toteuttaa uutta collapse-systeemiä.
- Oletuksena collapsoidutkin rajaukset pitäisi näyttää avattuina ainakin silloin, kun muokataan hakua, jossa on tehty ko. rajauksia.
- Myös EDS:n tarkennetussa haussa on rajaimia, pitäisi olla yhtenäinen.
- Huolettaa, että esim. usein tiettyä rajainta käyttävää käyttäjää alkaa ärsyttää, jos avauksia ei muisteta ja aina pitää klikata rajain auki ensin.
- Huolettaa, että nyt ei enää näe nopealla silmäyksellä, millaisia vaihtoehtoja on tarjolla.
- Huolettaa, että tästä tulee aika iso muutos hyötyyn nähden.
| <span class="label-block form-inline"><?=$this->transEsc($params->getFacetLabel($current['field']))?>:</span> | ||
| </div> | ||
| <div> | ||
| <button class="list-group-item title collapsed" type="button" data-toggle="collapse" data-target="#<?=$current['field']?>" controls="#<?=$current['field']?>" aria-expanded="false"> |
There was a problem hiding this comment.
controls-attribuutilla ei kuulu olla risuaitaa
| <div class="solr-adv-filter <?=$this->escapeHtmlAttr(str_replace(' ', '', $this->field))?>-container"> | ||
| <label id="limit_<?=$this->escapeHtmlAttr(str_replace(' ', '', $this->field))?>"><?=$this->transEsc($this->list['label'])?>:</label> | ||
| <div class="list-group facet"> | ||
| <button class="list-group-item title collapsed" type="button" data-toggle="collapse" data-target="#<?=$this->escapeHtmlAttr(str_replace(' ', '', $this->field))?>" controls="<?=$this->escapeHtmlAttr(str_replace(' ', '', $this->field))?>" aria-expanded="false"> |
There was a problem hiding this comment.
Ota siistitty field talteen ensin, niin ei tarvitse tehdä samaa str_replace:a moneen kertaan.
|
Olen kyllä samaa mieltä Eren kanssa että valinnat aiheuttaa lisäpainalluksia. Auttaisi jos käyttäjän avaamaat rajaimet muistettaisiin. Collapsointi on katsottu hyödylliseksi jos tarkennettuun hakuun tuodaan lisää rajaimia (mediatyppi, käyttöoikeudet jne.). Jos rajaimia on käytössä muutama, niin collapsointiin ei mielestäni ole tarvetta. 
Tekstityyli saisi olla yhtenevä (nyt ylhäällä on isolla ja alhaalla pienellä kirjaimella) |
No description provided.