Summary2: mulighet til å bruke "hideEmptyFields" og "hidden" når "type"="layoutSet"

[Hedda-ro]

Description

Hei!

BR/Årsregnskap har 11 ulike skjemaer med en rekke komponenter i hvert skjema, som gjør det nødvendig å bruke "type"="layoutSet" for Summary2-komponenten/visningen. Dette kan innebære en kombinasjon av rundt 40 komponenter i et hovedskjema, og mer enn 10 unike "Grid"-komponenter i et underskjema (med flere hundrevis av input-felter).

Et viktig behov hos oss nå er muligheten til å skjule "tomme felter" i sammendraget (og tilsvarende i PDF-visningen). Dette for å unngå unødvendig mye støy.

Per nå fungerer "hideEmptyFields" når "type"="component", men vi har behov for at "hideEmptyFields" (samt "hidden") fungerer når "type"="layoutSet" 🙏

Eksempelet her viser et lite utdrag fra ett underskjema (er typisk i underkant av 10 felter i underskjema som er obligatoriske, blant hundrevis av inputfelter totalt sett). Ved å kunne skjule tomme felter sikrer man at man unngår unødvendig støy.

Additional Information

Relatert feature request: 3159

[olemartinorg]

Tenker denne er relatert til:

• Toggle for å automatisk skjule tomme rader i oppsummeringselement #2471

[Hedda-ro]

Absolutt, er muligens litt unødvendig at vi har 3 (4) åpne, relaterte issues ifb. Summary2 nå:

• 2471
• 3153
• Nice to have: 3159

Behovet vårt nå er litt mer spisset enn det som kom frem tidligere forrige høst (nå: ønsker å skjule alle tomme felter, og det kan være lik visning i sammendraget og PDF). Vi bruker jo mye Grid (som vi er så glad i), forstår det slik den er muligens tricky mtp. ønsket her.

Blir kanskje et møte på dette fremover ?

[olemartinorg]

@Hedda-ro Jeg har begynt å jobbe med denne og prøver å kartlegge. Tror jeg fant problemet med #3234, og jobber med å få på plass en fiks der, men det slår meg jo at denne saken handler om mye av det samme.

Jeg mener riktignok jeg får hideEmptyFields til å fungere selv om jeg går via type = layoutSet, og undres om det dette egentlig handler om er at dere tenker hideEmptyFields burde skjult rader i Grid hvor alle komponentene i den raden er ubesvarte? Ut ifra skjermbildet virker det iallefall slik på meg, da den vel avbilder Grid-komponenten samt en overskrift.

Funksjonaliteten rundt hideEmptyFields var nok i utgangspunktet ikke tenkt brukt i Grid, da det kun var snakk om å ikke oppta mye plass med "Du har ikke lagt inn informasjon her"-meldinger i komponenter man ikke har besvart. Mens i Grid står det (som i bildet) bare "-" når man ikke har besvart, og selve raden er jo konfigurert opp på en slik måte at koden vår ikke alltid kan vite helt sikkert om hele raden bare kan skjules fordi komponentene inni er "tomme". Det er også litt utfordrende å innføre ny funksjonalitet som skjuler data som ikke har blitt skjult før, så her må vi nok lage enda et flagg som må skrus på..

Men hvis min antakelse her stemmer, hvordan tenker dere rundt header-radene? Har dere noen rader som bare inneholder (mellom-)overskrifter/tekster, som beskriver radene under? Isåfall er det jo også litt problematisk om vi bare fjerner tomme rader. Som for eksempel:

Inntekt	NOK
Varesalg	123,-
Utgift	NOK
Husleie	500,-
Strøm	200,-

Hvis det ikke var noe data under 'varesalg' her, ville vi da skjult hele den raden, men da ville man jo hatt overskriften 'Inntekt' stående alene uten noe data under seg. Som i #3159 må vi jo da finne en mekanisme for å også la dere skjule den raden som ikke lengre har noen innholdsrader å være overskrift for. Eller er ikke dette et case for dere? 🤔

[Hedda-ro]

Herlig!

Som du skriver, denne saken handler om mye av det samme som er nevnt i #3234, ja.

Hm, forsøker å kategorisere svaret her litt, bare gi en lyd om noe blir uklart. De layoutset'ene som vi har i skjemaene består av en rekke ulike komponenter samt nivå: enkeltvis eller gruppert (gruppert i form av Grid, Group eller Repeating Group).

Summary2 v/layoutset
Hm, ifb. Med "hideEmptyFields" for et helt layoutset - vil du teste f.eks. å alternativt legge inn dette:

{
"id": "Summary2-H-03-Subform",
"type": "Summary2",
"hideEmptyFields": true,
"target": {
"type": "layoutSet",
"id": "Underskjema",
"taskId": "Utfylling"
},

ui/Signering/layouts/01signering.json (linje 311) - Repo: vanlig årsregnskap

I dette tilfellet vises ingenting fra underskjema hos meg, selv om noen av komponentene i underskjema har verdier. Det vi også ser ved Summary2 - er at uavhengig om vi definerer "id": "Underskjema" eller "id": "Hovedskjema" ved "type": "layoutSet", så blir det hentet komponenter fra både underskjema og hovedskjema (at det skjer for "id": "Hovedskjema" er forståelig mtp. Subform-komponenten).

Summary2 v/Group:
At hideEmptyFields fungerer for en Group (#3234), vil delvis løse problemstillingen vi har - absolutt.

Summary2 v/Grid:
Yes, riktig antakelse! Vi ønsker at det fungerer for Grid, som du skriver. Dette er ikke et absolutt krav for å gå i produksjon, men UX-messig håper vi på det ifb. Å redusere visuelt støy for sluttbruker - spesielt i tilfellene hvor det er vedlagt konsernregnskap.

Du tolker også behovet riktig for Grid (beklager at det ikke var tydelig presisert ovenfor): ja, vi ønsker å skule rader hvor alle komponentene i den raden er ubesvarte i Grid.

Videre er det et godt spørsmål ifb. Tabelltitler. Vi tok en liten avklaring internt her, og ønsket er å skjule tabelltitler dersom radene under er tomme.

En ting som muligens er litt uheldig i vår Grid-konfigurasjon per nå, er at "headers" i Grid blir brukt både for å 1) vise tabelltitler (altså formålet med header) i tillegg til 2) bedre visualisering av enkelte poster - f.eks. en sum-post, som "Sum langsiktig gjeld" vist her, kan være "ekstra" relevant å fremheve for en sluttbruker:

Dersom dette med skjuling av tabelltittel-rader skal fungere basert på at radene under er tomme, så vil jeg tro vi muligens må gå vekk fra denne konfigurasjonen vi har per nå med headers (altså at den blir brukt til to ulike formål)?

Summary2 v/Repeating Group
Ikke nevnt eksplisitt i issuet her - men relevant i forhold til et layoutSet. I hovedskjemaer har vi to 'Repeating Groups' (som ikke er obligatoriske), men som inneholder input-felter som er obligatoriske - så sant input-feltetene blir opprettet (dette for å kjøre valideringer/logikk korrekt i backend). Akkurat disse tilfellene er muligens spesialtilfeller, som vi må regne med å vise uavhengig av om de er tomme eller ikke. Men vi har også en rekke 'Repeating Groups' i underskjemaet til Vanlig Årsregnskap (skjemaet som er nevnt som eksempel lenger oppe) - som ikke er obligatoriske. Per nå får jeg ikke testet det i tilknytning til "hideEmptyFields", men skal følge det opp fremover.

Ble mye her, bare gi en lyd om noe er uklart eller lignende. Takk for at du ser på det! 🤩

[olemartinorg]

Takk for et utfyllende og bra svar! 🙌

I dette tilfellet vises ingenting fra underskjema hos meg, selv om noen av komponentene i underskjema har verdier.

Okei, da er løsningen jeg jobber med på sporet iallefall. Jeg får opp noe her da, men forsåvidt heller ikke alt jeg trodde jeg skulle få opp. Jeg må nok grave videre i dette for å få løsningen helt optimal da.

Det vi også ser ved Summary2 - er at uavhengig om vi definerer "id": "Underskjema" eller "id": "Hovedskjema" ved "type": "layoutSet", så blir det hentet komponenter fra både underskjema og hovedskjema (at det skjer for "id": "Hovedskjema" er forståelig mtp. Subform-komponenten).

Det er litt uheldig ja, men jeg tror det er slik det er tenkt også. Når du sier "type": "layoutSet" tolker vi det som "hele layout-settet for den task-IDen du har spesifisert, eller gjeldende task om ingen er spesifisert". Attributtet id har ingen betydning der, så det holder med { "type": "layoutSet", "taskId": "Utfylling" }. Jeg noterer meg at det kan hjelpe med en litt mer spissing av valgmulighetene her, slik at man ikke lures til å gå i den fella.

Du kan forsåvidt også spesifisere at du vil vise bare en komponent for en annen taskId, hvis du bare ville ha oppsummering for underskjemaene:

"target": { 
  "type": "component",
  "id": "H-03-Subform",
  "taskId": "Utfylling"
}

Dersom dette med skjuling av tabelltittel-rader skal fungere basert på at radene under er tomme, så vil jeg tro vi muligens må gå vekk fra denne konfigurasjonen vi har per nå med headers (altså at den blir brukt til to ulike formål)?

Hmm, jeg er ikke nødvendigvis helt sikker på det. Om jeg skjønte det riktig har dere noen rader som bare er markert som header og har tekst i seg, mens andre rader også har komponenter i seg. Det sitter langt inne å lage funksjonalitet som skjuler andre komponenter om noen er tomme, så jeg er skeptisk til å gå den veien. Et alternativ (som jeg også ser for meg mtp. #3159) er å gi dere muligheten for å spesifisere dette selv ved hjelp av uttrykk, f.eks. slik at dere må lage et uttrykk som da skjuler en header-rad (dog bare i oppsummeringen) dersom alle cellene under den header-raden er tomme. Det er iallefall i de baner jeg tenker, for det er litt risikabelt å lage noe magi som skjuler header-rader basert på innhold i celler under - Grid-komponenten kan fint settes opp med en layout som ville blitt "ødelagt" av slik funksjonalitet.

Summary2 v/Repeating Group

Jepp, det du beskriver her vil jo føre til at vi viser oppsummeringen for disse feltene uansett om man har satt opp hideEmptyFields, - de vises jo uansett så lenge de er påkrevde. Men om oppsummeringen vises i signeringstasken gjør det kanskje ikke så mye? Man kan ikke sende inn første task uansett hvis man har unnlatt å fylle ut obligatoriske felter, så da må man eventuelt gå tilbake og slette raden man ikke skulle fylle ut noe i.