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

Tasklist-app by Daniel #444

Open
wants to merge 24 commits into
base: master
Choose a base branch
from
Open

Tasklist-app by Daniel #444

wants to merge 24 commits into from

Conversation

dannebrob
Copy link

Copy link

@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 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;

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.

@@ -1,13 +1,110 @@
body {
/* Box sizing rules */

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 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;
}

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