altinn-studio
altinn-studio copied to clipboard
12438 change name and design for tilgangsstyring tab in settings modal
Description
- Updated title text to be
Oppstartskontroll
instead ofTilgangsstyring
Both in tab list and the actual title of the tab content. - Removed "Aktørkrav" * and added tabel to render checkboxes inside it.
- Added logic to make sure that at least one checkbox should be choosen.
- Added a
modal
to show warning message when user tries to uncheck the last checked checkbox. - Added
helpText
and docslink
. - Updated and added some tests.
Related Issue(s)
- #12438
Verification
- [ ] Your code builds clean without any errors or warnings
- [ ] Manual testing done (required)
- [ ] Relevant automated test added (if you find this hard, leave it and we'll help out)
Documentation
- [ ] User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 89.93%. Comparing base (
b2168a7
) to head (8bb075b
). Report is 3 commits behind head on main.
Additional details and impacted files
@@ Coverage Diff @@
## main #12511 +/- ##
==========================================
+ Coverage 89.90% 89.93% +0.02%
==========================================
Files 1287 1291 +4
Lines 18365 18395 +30
Branches 2236 2239 +3
==========================================
+ Hits 16511 16543 +32
+ Misses 1595 1593 -2
Partials 259 259
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
I think to fix the tests after we finish updating AccessControlTab
component.
Overskriftforslag @standeren og @Annikenkbrathen:
- Hvem kan starte denne appen?
- Hvem eier og kan starte denne appen?
- Velg hvem som skal eie og kan starte denne appen
I og med at vi har en beskrivende tekst under som forklarer det med "eie" så holder jeg en knapp på den enkleste varianten i nr. 1
Bruk av spørsmål i overskrifter må eventuelt harmoniseres med hva vi ellers gjør innenfor samme funksjonalitetsområde. Hvis vi ikke skal ha spørsmål, kan 1 også være "Velg hvem som kan starte appen"
Overskriftforslag @standeren og @Annikenkbrathen:
- Hvem kan starte denne appen?
- Hvem eier og kan starte denne appen?
- Velg hvem som skal eie og kan starte denne appen
I og med at vi har en beskrivende tekst under som forklarer det med "eie" så holder jeg en knapp på den enkleste varianten i nr. 1
Bruk av spørsmål i overskrifter må eventuelt harmoniseres med hva vi ellers gjør innenfor samme funksjonalitetsområde. Hvis vi ikke skal ha spørsmål, kan 1 også være "Velg hvem som kan starte appen"
Jeg er enig i den enkleste varianten; og kanskje uten spm --> Velg hvem som kan starte appen
😄
Something wierd with the css here making the left menu scrollable 🤔 https://github.com/Altinn/altinn-studio/assets/71079896/c693f1e7-80c1-4104-ac04-3bac37789f95
But it beginning to look good! 💯
Regarding the scroll, it looks like the problem should be fixed for the entir LeftNavigationstab
, ? I have tested in dev and the scroll is also exists in tilgangsregler
tab. so maybe this should be fixed in a seperate issue to fix all tabs toghether. ?
https://github.com/Altinn/altinn-studio/assets/90609090/c087ede4-d071-45ce-91a4-22917996b07e