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

Käännöstoive: asiakastakaajan lisäys ja select-nappi #1014

Closed
IrinaHalminen opened this issue Jan 16, 2024 · 19 comments
Closed

Käännöstoive: asiakastakaajan lisäys ja select-nappi #1014

IrinaHalminen opened this issue Jan 16, 2024 · 19 comments
Assignees

Comments

@IrinaHalminen
Copy link

Tukipyyntö

Asiakkaalle takaajaa/huoltajaa lisätessä klikataan Lisää takaaja. Tästä avautuu ponnahdusikkuna, jossa voi etsiä takaajaksi lisättävän asiakkaan. Tuloksena saatavien asiakastietojen perässä oleva valintanappi on englanniksi (Select). Saako muutettua suomeksi?

Jos on mitenkään mahdollista, voisiko valintanapin siirtää asiakastietojen eteen ennen Kortti-saraketta, koska nyt se jää ponnahdusikkunassa oikealle ja näytön resoluutiosta riippuen joskus piiloon (vaatii oikealle skrollausta). Voin tehdä tästä uuden tiketin, jos on toteutettavissa tai nähdään lainkaan tarpeelliseksi?

Molemmista asioista tullut palaute asiakaspalvelusta.

Jotain muuta?

No response

@AnneliO
Copy link
Contributor

AnneliO commented Jan 16, 2024

Mie luulen, että nyt jokin Koha-Suomi-muutos kadottaa käännöksen tuosta Select-napista, koska outi-nextillä (jossa ei ole vielä juurikaan muutoksia) nappi on kääntynyt ihan oikein. IntranetUserJS-asetuksen tyhjentäminen ei vaikuttanut asiaan, joten ongelma ei johdu ainakaan niistä.

Hakuikkunalle voisi tehdä periaatteessa saman tempun kuin mitä Z39.50-hakuikkunalle tehdään JS-rimpsulla eli ikkunaa muutetaan isommaksi, jolloin tiedot mahtuvat paremmin.

Sain periaatteessa toimivan version tehtyä tuon pohjalta, mutta joku paremmin JavaScriptiä tunteva voisi varmistaa rimpsun vielä:

/// ALKU ///
/* Muuta asiakashakuikkunan koko suuremmaksi. Ikkunan koko on turhan pieni hakutuloksille. */
//Members - search patrons pop-up
 //BEGIN Resize Patron search pop-up
  if (document.location.href.indexOf('search.pl')>-1) window.resizeTo((screen.width * 0.55), (screen.height * 0.9)), window.moveTo(0, 0);
  $(window).on('load resize', function(){
   $('#common_patron_search.common').height($(this).height() * 0.75);
   $('#common_patron_search.common').css('overflow-y', 'scroll');
  });
//END

/// LOPPU ///

@AnneliO
Copy link
Contributor

AnneliO commented Jan 16, 2024

Nyt kun tutkin tarkemmin, niin version 22.xx käännöksissä ei itse asiassa ole käännöstä tuolle Select-sanalle. Tai on, mutta sitä ei ole kytketty tuohon asiakashakuun. Versiossa 23.xx käännös on kytketty asiakashakuun, joten sen kääntyy oikein. Eli tuo ongelma korjaantuu versionvaihdossa. En käyttäisi siihen nyt tällä hetkellä enempää aikaa. :)

@atarnus
Copy link

atarnus commented Jan 16, 2024

Katselin tuon taulukon muodostumista, niin hoksasin, että siellähän on vähän hassusti kirjastotieto kahteen kertaan:
image

Tuon pop-up-ikkunan kun avaa, niin sinne tulevat sarakkeet näkyivät muodostuneen URL:issa:
cgi-bin/koha/members/search.pl?columns=cardnumber,name,category,branch,dateofbirth,address-library,action

members.js:stä tämä löytyy, ja vaihtaisin näin: actionin alkuun ja address-libraryn tilalle pelkkä address:

$('body').on('click', '#guarantor_search', function(e) {
    e.preventDefault();
    var newin = window.open('/cgi-bin/koha/members/search.pl?columns=action,cardnumber,name,category,branch,dateofbirth,address','popup','width=1024,height=768,resizable=no,toolbar=false,scrollbars=yes,top'>
});

Ja lopputulos näyttäisi tältä:
guarantor

Tuohon ikkunan kokoon ei varmaan tarvi puuttua jos se select nappula tulee alkuun?

@atarnus
Copy link

atarnus commented Jan 16, 2024

Sarakkeiden leveyteen vaikuttaa tosiaan mahdollisen sisällön leveys: nimi-, asiakastyyppi- ja kirjasto-kentät eivät katkaise sisältöä, joten taulukon leveys tulee vaihtelemaan paljon.

Huomasin myös, että nuo hakukentätkään ei ole kääntyneet kokokaan, niin tarkistin, ja nextillä näyttävät nekin kääntyvän.
image

@AnneliO
Copy link
Contributor

AnneliO commented Jan 22, 2024

Koha-Suomen palaveri 22.1.2024: Tehdään yhteisön Bugzillaan kaksi tikettiä:

  • Valinta-napin siirrosta rivin alkuun ja tarjotaan sinne korjaus. Ei tuoda itselle väliaikaisesti ennen kuin nähdään, päätyykö muutos yhteisöön.
  • Muutos, että osoite-sarakkeeseessa ei turhaan näytetä toiseen kertaan kirjasto-tietoa.

@AnneliO
Copy link
Contributor

AnneliO commented Mar 25, 2024

@IrinaHalminen teetkö Bugzillaan tiketin Valinta-napin siirrosta rivin alkuun? :)

@IrinaHalminen
Copy link
Author

Kiitos täggäyksestä, jäinkin pohtimaan, kenen työksi tiketti tuli. Nyt laitettu pienoisella viiveellä: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=36558

@atarnus
Copy link

atarnus commented May 14, 2024

Tämän saisi muuten kyllä ihan näppärästi JS-rimpsuunkin, jos halutaan.

$(document).on('click', '#guarantor_search', function(e) {
    e.preventDefault();
    var newin = window.open('/cgi-bin/koha/members/search.pl?columns=action,cardnumber,name,category,branch,dateofbirth,address','popup','width=1024,height=768,resizable=no,toolbar=false,scrollbars=yes,top');
});

Tuo ajaa yli sen alkuperäisen funktion.

@atarnus
Copy link

atarnus commented May 14, 2024

Yhteisössä on nyt kirjoitettu tuo popup-ikkunan avautuminen bootstrap-moduuliin, niin ei fiksaannu enää tuolla koodinpätkällä. Ja koska minulla ei ole toimivaa yhteisöversiota, niin on aika tyhjää koittaa rukata siihen mitään pätsiä. Kävin sandboxilla katsomassa, niin taulu on edelleen samanlainen moduulin sisällä, mutta pääkäyttäjien lienee tarpeen testata meneekö se valinta-nappi piiloon eri tilanteissa, ja halutaanko se ensimmäiseen sarakkeeseen.

@sksandel
Copy link

sksandel commented May 31, 2024

Testeillä näyttäisi toimivan nyt kuten speksattu. Valitse-nappi kätevästi ensimmäisessä sarakkeessa ja ylimääräinen kirjastotieto poissa. En keksinyt missä tilanteessa valinta-napin pitäisi "mennä piiloon". Lisää takaaja-toiminto oli näkyvissä vain niillä asiakastyypeillä, joille takaaja kuuluu lisätä.
image

@CodoDerDritte
Copy link
Collaborator

CodoDerDritte commented May 31, 2024

Yhteisöversion muutoksista johtuen korjaus ei tule olemaan kestävä. Homma on samalla tapaa rikki taas seuraavan versiopäivityksen jälkeen nyt tehdystä korjauksesta huolimatta. Muutos on lähinnä "olisihan se kiva" osastoa ja luonteeltaan lähinnä kosmeettinen. Mikäli korjaus kuitenkin halutaan, voitte lisätä Kassun laittaman JavaScript-rimpsun Kohaan pitäen mielessä korjauksen tilapäisyyden. Merkitsen tämän tiketin 'Odottaa' tilaan ja tutkitaan tilanne uudestaan seuraavan versiopäivityksen jälkeen.

@johannaraisa
Copy link

Koha-Suomen kehittäjien viikkopalaveri 3.6.2024: Tehdään tästä js-liitännäinen.

@lmstrand
Copy link
Collaborator

lmstrand commented Jun 4, 2024

Viety testeille testattavaksi plugin IntranetUserJS: Add guarantor select view fixes.

@sksandel
Copy link

sksandel commented Jun 4, 2024

Vaskin testeillä testattu. Toimii. Samat havainnot kuin aiemmassa toteutuksessa.

Piiloon tuo nappi ei mene, vaikka hakutuloksessa olisi asiakas, joka ei voi olla takaaja (esim automaatti). Mutta ei se aikaisemminkaan ole mennyt.

@Kati-Helle
Copy link

Kati-Helle commented Jun 12, 2024

Helle-testissä Lisää takaaja -asiakashaun hakutuloksessa

  • Valitse-painike ensimmäisenä
  • Osoite-sarakkeessa ei enää toisteta tarpeettomasti kirjaston nimeä
    kuva

@IrinaHalminen
Copy link
Author

Vaarassa testattu ja samat havainnot kuin edellä. Jos hakee tyhjällä kentällä, virheilmoitus on englanniksi. Onko tarvetta muuttaa (aika vähän varmaan näin hakuja tapahtuu)?

image

@AnneliO
Copy link
Contributor

AnneliO commented Jun 14, 2024

Hmm.. tuolle Please fill out this field -tekstille on kyllä käännös olemassa fi-FI-staff-prog.po-tiedostossa:

#. SCRIPT
#: koha-tmpl/intranet-tmpl/prog/en/includes/patron-search.inc:254
msgid "Please fill out this field."
msgstr "Kirjoita kenttään."

@AnneliO
Copy link
Contributor

AnneliO commented Jun 17, 2024

Vaarassa testattu ja samat havainnot kuin edellä. Jos hakee tyhjällä kentällä, virheilmoitus on englanniksi. Onko tarvetta muuttaa (aika vähän varmaan näin hakuja tapahtuu)?

image

Teetkö Irina tästä erillisen tiketin, niin jatketaan sen selvittelyä siinä. :)

@IrinaHalminen
Copy link
Author

IrinaHalminen commented Jun 18, 2024

Teen toisen tiketin em. asiasta -> #1295 Tämä toimii tuotannossa, kiitos! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Suljettu
Development

No branches or pull requests

8 participants