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

Lisätty palveluohjaukselle mahdollisuus muuttaa hakemuksen hakutoiveiden järjestystä #6171

Merged

Conversation

msavolainen-gofore
Copy link
Collaborator

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.

Screenshot 2024-12-19 at 14 24 09

@msavolainen-gofore msavolainen-gofore added the enhancement Uusi toiminnallisuus tai parannus label Dec 19, 2024
<Button
appearance="inline"
icon={faArrowUp}
text={i18n.documentTemplates.templateEditor.moveUp}
Copy link
Contributor

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.

Copy link
Contributor

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(
Copy link
Contributor

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ä:
Suggested change
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)))

@msavolainen-gofore msavolainen-gofore force-pushed the application-employee-reorder-unit-preferences branch from 0febfce to 7cbe86f Compare December 23, 2024 05:54
@msavolainen-gofore msavolainen-gofore merged commit 2db1167 into master Dec 23, 2024
28 of 29 checks passed
@msavolainen-gofore msavolainen-gofore deleted the application-employee-reorder-unit-preferences branch December 23, 2024 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Uusi toiminnallisuus tai parannus
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants