Skip to content

HW1 updated#29

Open
RostislavKorst wants to merge 1 commit into
obabichev:homework/rkorstfrom
RostislavKorst:homework/rkorst
Open

HW1 updated#29
RostislavKorst wants to merge 1 commit into
obabichev:homework/rkorstfrom
RostislavKorst:homework/rkorst

Conversation

@RostislavKorst
Copy link
Copy Markdown

Здравствуйте!
Я немного поправил код для 1 дз,чтобы убрать хардкод. Однако я не понимаю, как лучше для задания отображать вложенные задачи(как лучше обрабатывать таски и сабтаски и какой компонент использовать). Могли бы вы, пожалуйста, подсказать в каком направлении дальше двигаться?

Comment thread public/index.html
rel="stylesheet"
href="https://cdnjs.cloudflare.com/ajax/libs/semantic-ui/2.4.1/semantic.min.css"
/>
<link rel="icon" href="../favicon.ico" />
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Я думаю, favicon.ico лучше также поместить в папку public, тогда при сборке эта иконка будет просто перекопирована со всей папкой public в собранное приложение

console.log(tasks);
const BoardTaskList = () => {
return (
<>
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Скорей всего в итоге Фрагмент здесь будет нужен, но здесь и сейчас, пока из BoardTaskList возвращается только массив элементов, Фрагмент скорее избыточен

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