-
Notifications
You must be signed in to change notification settings - Fork 12
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
Lisätty palveluohjaukselle mahdollisuus muuttaa hakemuksen hakutoiveiden järjestystä #6171
Lisätty palveluohjaukselle mahdollisuus muuttaa hakemuksen hakutoiveiden järjestystä #6171
Conversation
<Button | ||
appearance="inline" | ||
icon={faArrowUp} | ||
text={i18n.documentTemplates.templateEditor.moveUp} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Vaikka dokumenttitemplateista löytyykin jo sopivat tekstit näille nappuloille, niin lisäisin uudet käännösavaimet sopivaan paikkaan i18n.application
alle näitä tomintoja varten.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Voisko tekstit jättää kokonaan pois, ja pitää vain ikonit? Nyt ehkä lihavoidut nappien tekstit korostuvat liikaa, kun varsinaiset siirrettävät asiat on ohuemmalla fontilla. Ennenhän siinä oli pelkästään "X Poista", mutta nyt kun siinä on noi nuolet myös, niin se on mielestäni aika selkeä mitä käyttäjä voi odottaa napeista tapahtuvaksi, ilman tekstejäkin. Jos tälle on virallinen design jo olemassa, niin sitten tietty voi mennä sillä.
icon={faArrowUp} | ||
text={i18n.documentTemplates.templateEditor.moveUp} | ||
onClick={() => | ||
setApplication( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tämä olisi helpommin luettava jos käyttäisi helper funktiota array slicauksen sijaan. Esim:
function swapElements<T>(array: T[], index1: number, index2: number): T[] {
const result = [...array]
const temp = result[index1]
result[index1] = result[index2]
result[index2] = temp
return result
}
käyttö tässä:
setApplication( | |
setApplication(set('form.preferences.preferredUnits', swapElements(preferredUnits, i, i - 1))) |
ja vastaavasti alaspäin nappulassa
onClick={() => setApplication(set('form.preferences.preferredUnits', swapElements(preferredUnits, i, i + 1)))
0febfce
to
7cbe86f
Compare
Ennen tätä muutosta
Tarve uudelleenjärjestää hakemuksen hakutoiveet on noussut esim puhelussa asiakkaan kanssa. Palveluohjaaja on joutunut poistamaan hakutoiveet ja lisäämään ne takaisin halutussa järjestyksessä.
Tämän muutoksen jälkeen
Työntekijän puolelle hakemuksen editointiin on lisätty kuntalaisen puolta vastaavat painikkeet uudelleenjärjestää hakutoiveet.