New review branch #3
Conversation
salierri
left a comment
salierri
left a comment
There was a problem hiding this comment.
Első TS projektnek igazán szép kód!
simpletodo/src/App.tsx
Outdated
| </p> | ||
| </header> | ||
| <div> | ||
| <Title /> |
There was a problem hiding this comment.
Javascriptben a 2 space az elfogadott konvenció.
Vannak általánosan elfogadott style guideok (https://github.com/airbnb/javascript), ezeket jó ismerni és ezalapján formázni a kódot, akkor egységesebb lesz a többi js fejlesztővel.
Még jobb, ha be van húzva az ESlint / TSlint, és be vannak állítva szabályok amire szól.
| document.getElementById('root') as HTMLElement | ||
| ); | ||
| root.render( | ||
| <React.StrictMode> |
There was a problem hiding this comment.
Van valami oka hogy nem szeretnéd a strict mode-ot?
There was a problem hiding this comment.
Nem most, de korábban problémát okozott a plusz 2 felesleges renderelés.
README.md
Outdated
|
|
||
| - Clone the repository using the command git clone `https://github.com/kovesdieszter/SimpleToDo.git`. | ||
| - Install all necessary packages asked by your IDE. | ||
| - Run `npm start` from `simpletodo` directory. |
There was a problem hiding this comment.
Fölösleges a projektnek még egy almappa, annak csak akkor van szerepe, ha több projektet is tárolsz egy repo-ban, ilyenkor mehet csak simán a gyökérbe.
There was a problem hiding this comment.
Ez a komplikált deploy miatt most kivételesen már így maradna.
| YEAR: 'rgba(51, 170, 51, .3)' | ||
| } | ||
|
|
||
| function setBackgroundColor() { |
There was a problem hiding this comment.
This also works as a one-liner with arrow expression
No description provided.