Skip to content

Styles was updated v2#3

Open
4s-ary wants to merge 2 commits intodevelopfrom
issue_search
Open

Styles was updated v2#3
4s-ary wants to merge 2 commits intodevelopfrom
issue_search

Conversation

@4s-ary
Copy link
Owner

@4s-ary 4s-ary commented Oct 23, 2021

image

@4s-ary 4s-ary linked an issue Oct 23, 2021 that may be closed by this pull request
@@ -0,0 +1,23 @@
import { FC } from "react";
Copy link
Collaborator

Choose a reason for hiding this comment

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

Путь должен быть assets/icons

)
const InteractionArea: FC<Props> = ({inputValue, handleInputCallBack, addButton, addButtonText}) => {
const handleTodoAddingOnKeyDown = (event: KeyboardEvent) => {
if (event.code === "Enter") {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Должен предусмотреть все enter на клавиатуре

}
};
return (
<div className={styles.Container}>
Copy link
Collaborator

Choose a reason for hiding this comment

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

стили с маленькой буквы styles.container

justify-content: center;
width: 40%;
margin: 0 auto;
height: fit-content;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Не нужно

return (
<div key={id} className={styles.listElement}>
{title}
<div className={styles.deleteItem} id={`${id}`} onClick={deleteListElementCallBack}>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Сделай так - onClick={() => deleteListElementCallBack(id)}


}

const Search: FC<SearchProps> = ({showSearchBox}) => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Условный рендер должен быть сверху, а не внутри компонента

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