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

Ylvas ToDo-project Week 11 #446

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

Conversation

YlvaKarlsson
Copy link

This was a fun week - finally!
Happy to be able to have a working project :D

Copy link

@majazimnoch majazimnoch left a comment

Choose a reason for hiding this comment

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

Great job Ylva, I'm impressed by how ambitious your approach to this project was! Everything looks great, the date is in a nice format, a lot of extra functions and functionality. Maybe you can try later to make the categories work, then I will be happy to learn how you made it work!

code/src/App.js Outdated
Comment on lines 20 to 21

// ctrl + c inside the terminal to stop the liveServer

Choose a reason for hiding this comment

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

Nice that you write such comments but I think our final code should be without comments, however, I strongly advise you to make a separate google sheet - docs - where you put all your coding notes ;)

Comment on lines 6 to 19
id: '189438fdjhrjejioe9845',
name: 'ToDo1',
isNew: false
},
{
id: '189438fdjhrjejioe9846',
name: 'ToDo2',
isNew: false
},
{
id: '189438fdjhrjejioe9847',
name: 'ToDo3',
isNew: false
}

Choose a reason for hiding this comment

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

you can also start your list with no to-dos by making an empty array, like:

const initialState = {
  items: [ ]
}

Comment on lines 8 to 12
Favicon provided free from Icons8 - here you can reed more about the usage of this in your own projects
https://icons8.com/license
Notice the use of %PUBLIC_URL% in the tags above.
It will be replaced with the URL of the `public` folder during the build.
Only files inside the `public` folder can be referenced from the HTML.

Choose a reason for hiding this comment

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

this is such a great thing to add credits in the html, commented out.

Comment on lines 8 to 33
const NewToDo = styled.form`
line-height: 0.5rem;
padding-bottom: 0.8rem;
`;

const Input = styled.input`
background: #a83256;
border: none;
padding: 8px;
font-size: 18px;
font-family: 'Baloo 2', cursive;
border-bottom: 2px dashed;
:focus {
outline: none;
}
`;

const AddButton = styled.button`
font-family: 'Baloo 2', cursive;
font-size: 18px;
border: none;
background: transparent;
cursor: pointer;
`;

export const AddToDo = () => {

Choose a reason for hiding this comment

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

nice that you like using styled-components! if you like different ways of styling you can read more about Sass or tailwind css :) Maybe you can also try to switch to rems, ems instead of px.

@@ -1,7 +1,12 @@
@import url('https://fonts.googleapis.com/css2?family=DynaPuff:wght@400;600&display=swap');

body {
background: rgb(51,102,255);
background: linear-gradient(180deg, rgba(51,102,255,1) 0%, rgba(255,0,255,1) 100%);

Choose a reason for hiding this comment

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

nice styling with gradient! ;)

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