Skip to content

Tasklist-app by Daniel#444

Open
dannebrob wants to merge 24 commits intoTechnigo:masterfrom
dannebrob:master
Open

Tasklist-app by Daniel#444
dannebrob wants to merge 24 commits intoTechnigo:masterfrom
dannebrob:master

Conversation

@dannebrob
Copy link
Copy Markdown

Copy link
Copy Markdown

@codeandjazz codeandjazz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hallå där Daniel!

Hoppas brödet blir gott och ungarna beter sig ;)

Jag har kollat igenom din kod och det verkar som att du har en bra uppfattning om React och Redux. Det var flera intressanta grejer i din index.css fil som jag blev nyfiken på, som jag aldrig sett förut. Jag gillar enkelheten i hur du har valt att utforma din hemsida och även att man kan "spara" sin att-göra lista genom local storage. Det var en bra touch.

Annars har jag inte så mycket att säga. Det var inga konstigheter, förutom den där buggen som nämnde som du säkert jobbar på fortfarande. Du kan ju alltid gå tillbaka och fixa den när du får lite tid över.

Snyggt jobbat & ha det bra.

//Tess

Comment thread code/src/index.css
Comment on lines +72 to +88
/* Remove all animations, transitions and smooth scroll for people that prefer not to see them */
@media (prefers-reduced-motion: reduce) {
html:focus-within {
scroll-behavior: auto;
}

*,
*::before,
*::after {
animation-duration: 0.01ms !important;
animation-iteration-count: 1 !important;
transition-duration: 0.01ms !important;
scroll-behavior: auto !important;
}
}
html {
scroll-behavior: smooth;
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

intressant! detta har jag aldrig sett förut.

Comment thread code/src/index.css
@@ -1,13 +1,110 @@
body {
/* Box sizing rules */
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bra kommentarer genom hela filen.

Comment thread code/src/index.css
Comment on lines +37 to +51
/* Set core root defaults */
html:focus-within {
scroll-behavior: smooth;
}

/* Set core body defaults */
body {
text-rendering: optimizeSpeed;
line-height: 1.5;
}

/* A elements that don't have a className get default styles */
a:not([className]) {
text-decoration-skip-ink: auto;
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hade varit kul att veta varför du gör alla dessa inställningar här. Vissa har jag aldrig sett förut. Vad är "core body defaults" tex? Låt oss ta ett snack om det en dag, så får jag lära mig något nytt.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants