Skip to content

Commit 123f288

Browse files
authored
chore(storefront): fix cards in mdx content (#2289)
fixes cards using wrong `color`. fixes margin and padding in cards after we started using our own components. fixes siteconfig having double of one page.
1 parent bbd8086 commit 123f288

File tree

5 files changed

+29
-65
lines changed

5 files changed

+29
-65
lines changed

apps/storefront/app/monstre/feilmeldinger/page.mdx

+7-23
Original file line numberDiff line numberDiff line change
@@ -153,14 +153,9 @@ Feltet må ha 3:1 kontrast til bakgrunnsfargen. Hvis vi også bruker rød tekst
153153
Vi viser feilmeldingen under feltet med feil. Dette er det vanligste mønsteret, selv om det også diskuteres om den skal stå mellom ledeteksten og feltet.
154154

155155
<Card
156-
color='first'
156+
color='brand1'
157157
>
158-
<CardContent
159-
style={{
160-
paddingTop: '0px',
161-
paddingBottom: '0px'
162-
}}
163-
>
158+
<CardContent>
164159
Merk: Vi ønsker å holde oss oppdatert om andre anbefalinger om plassering, og har [pågående diskusjon om dette på Git](https://github.com/digdir/designsystemet/discussions/1684#discussioncomment-9339006).
165160

166161
</CardContent>
@@ -250,16 +245,10 @@ Vi kan bruke disse aria-attributtene og rollene:
250245
- Vi bruker `aria-describedby` for å koble feilmelding til feltet.
251246

252247
<Card
253-
color='first'
248+
color='brand1'
254249
>
255-
<CardContent
256-
style={{
257-
paddingTop: '0px',
258-
paddingBottom: '0px'
259-
}}
260-
>
250+
<CardContent>
261251
Vi unngår inntill videre å bruke `aria-errormessage` da den ikke har full støtte av hjelpemidler per nå. Men vi kommer til å oppdatere retningslinjene om støtten blir bedre i fremtiden. [Se diskusjon på github](https://github.com/digdir/designsystemet/discussions/1684)
262-
263252
</CardContent>
264253
</Card>
265254

@@ -286,18 +275,13 @@ For feilmeldinger som dukker opp dynamisk må vi bruke `aria-live` for at meldin
286275
<br />
287276

288277
<Card
289-
color='third'
278+
color='brand3'
290279
>
291-
<CardContent
292-
style={{
293-
paddingTop: '0px',
294-
paddingBottom: '0px'
295-
}}
296-
>
280+
<CardContent>
297281
Retningslinjene er utarbeidet i en tverretatlig arbeidsgruppe med deltakere fra Digdir, Nav, Skatt, Brreg, Politiet, KS DIF og Oslo kommune. Du kan påvirke arbeidet i [Github](https://github.com/digdir/designsystemet/discussions/1684) eller i [#Mønster-kanalen](https://designsystemet.slack.com/archives/C05RBGB92MC/p1712751837722749)[Slack](https://join.slack.com/t/designsystemet/shared_invite/zt-2438eotl3-a4266Vd2IeqMWO8TBw5PrQ).
298-
299282
</CardContent>
300283
</Card>
284+
301285
<Contributors
302286
authors={[
303287
'Roy Halvor Frimanslund (Brreg)',

apps/storefront/app/monstre/obligatoriske-og-valgfrie-felt/page.mdx

+12-16
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@ import { MenuPageLayout } from '@layouts';
44
import { Image } from '@components';
55

66
export const metadata = {
7-
title: 'Validering og feilmeldinger',
8-
description: 'Slik forteller du brukeren at noe har gått galt eller mangler.',
7+
title: 'Obligatoriske og valgfrie skjemafelt',
8+
description: 'Det er flere måter å markere obligatoriske felt på som oppfyller kravene til merking.',
99
};
1010

1111
export default ({ children }) => (
@@ -20,20 +20,6 @@ export default ({ children }) => (
2020

2121
Det er flere måter å markere obligatoriske felt på som [oppfyller kravene til merking](https://www.uutilsynet.no/veiledning/skjema/38#ledetekster_og_instruksjoner). I dag gjør vi dette ulikt på tvers. Noen bruker asterisk (stjerne), noen bruker ord, andre informerer i forkant om hva som må fylles ut. Klarer vi å gjøre dette mer likt på tvers blir det lettere for innbygger å forstå og kjenne igjen mønsteret på tvers av løsningene våre. Det vil alltid være unntak og ulike kontekster som krever ulik merking.
2222

23-
<Card
24-
color='third'
25-
>
26-
<CardContent
27-
style={{
28-
paddingTop: '0px',
29-
paddingBottom: '0px'
30-
}}
31-
>
32-
*Retningslinjene er utarbeidet i en tverretatlig arbeidsgruppe med deltakere fra Digdir, Nav, Skatt, Brreg og Oslo Origo. Du kan påvirke arbeidet i [Github](https://github.com/digdir/designsystemet/issues/new) eller i [#Mønster-kanalen](https://designsystemet.slack.com/archives/C05RBGB92MC/p1712751837722749)[Slack](https://join.slack.com/t/designsystemet/shared_invite/zt-2438eotl3-a4266Vd2IeqMWO8TBw5PrQ).
33-
34-
</CardContent>
35-
</Card>
36-
3723
En generell retningslinje er at vi bør unngå å be om informasjon vi ikke trenger, altså unngå valgfrie felt.
3824

3925
Her tar vi for oss 3 eksempler.
@@ -98,3 +84,13 @@ En kombinasjon av obligatoriske og valgfrie felt er ikke ideellt! Men det vil v
9884
- [UU-tilsynet](https://www.uutilsynet.no/veiledning/skjema/38#ledetekster_og_instruksjoner)<br/>
9985
- [Gov UK - Question pages](https://design-system.service.gov.uk/patterns/question-pages/)
10086
- [NN Group - Marking Required Fields in Forms](https://www.nngroup.com/articles/required-fields/)
87+
88+
<br />
89+
90+
<Card
91+
color='brand3'
92+
>
93+
<CardContent>
94+
*Retningslinjene er utarbeidet i en tverretatlig arbeidsgruppe med deltakere fra Digdir, Nav, Skatt, Brreg og Oslo Origo. Du kan påvirke arbeidet i [Github](https://github.com/digdir/designsystemet/issues/new) eller i [#Mønster-kanalen](https://designsystemet.slack.com/archives/C05RBGB92MC/p1712751837722749)[Slack](https://join.slack.com/t/designsystemet/shared_invite/zt-2438eotl3-a4266Vd2IeqMWO8TBw5PrQ).
95+
</CardContent>
96+
</Card>

apps/storefront/app/monstre/systemvarsler/page.mdx

+2-8
Original file line numberDiff line numberDiff line change
@@ -19,16 +19,10 @@ export default ({ children }) => (
1919
);
2020

2121
<Card
22-
color='third'
22+
color='brand3'
2323
>
24-
<CardContent
25-
style={{
26-
paddingTop: '0px',
27-
paddingBottom: '0px'
28-
}}
29-
>
24+
<CardContent>
3025
*Retningslinjene er under arbeid fra 5. juni 2024 i en tverretatlig arbeidsgruppe med deltakere fra Digdir, Nav, Skatt, Brreg, Politiet, KS DIF og Oslo kommune. Alle er velkommen til å påvirke arbeidet i [Github](https://github.com/digdir/designsystemet/discussions/2083) eller i [#Mønster-kanalen](https://designsystemet.slack.com/archives/C05RBGB92MC/p1712751837722749)[Slack](https://join.slack.com/t/designsystemet/shared_invite/zt-2438eotl3-a4266Vd2IeqMWO8TBw5PrQ).
31-
3226
</CardContent>
3327
</Card>
3428

apps/storefront/components/MdxContent/MdxContent.module.css

+8
Original file line numberDiff line numberDiff line change
@@ -134,3 +134,11 @@
134134
.content > *:last-child {
135135
margin-bottom: 0;
136136
}
137+
138+
.content [class~='ds-card'] p {
139+
margin: 0;
140+
}
141+
142+
.content [class~='ds-card'] {
143+
margin: var(--ds-spacing-3) 0;
144+
}

apps/storefront/siteConfig.tsx

-18
Original file line numberDiff line numberDiff line change
@@ -164,24 +164,6 @@ export const SiteConfig = {
164164
},
165165
],
166166
},
167-
{
168-
name: 'Kommende',
169-
url: 'monstre/skjema',
170-
children: [
171-
{
172-
name: 'Systemvarsler *',
173-
url: 'monstre/systemvarsler',
174-
},
175-
{
176-
name: 'Dato',
177-
url: 'monstre/dato',
178-
},
179-
{
180-
name: 'Obligatoriske felt *',
181-
url: 'monstre/obligatoriske-og-valgfrie-felt',
182-
},
183-
],
184-
},
185167
{
186168
name: 'Kommende',
187169
url: 'monstre/skjema',

0 commit comments

Comments
 (0)