Skip to content
This repository has been archived by the owner on Feb 15, 2022. It is now read-only.

[feature] update app design #77

Closed
TFK70 opened this issue Apr 30, 2021 · 4 comments
Closed

[feature] update app design #77

TFK70 opened this issue Apr 30, 2021 · 4 comments
Assignees

Comments

@TFK70
Copy link

TFK70 commented Apr 30, 2021

С чем связан запрос на фичу?

Обновить дизайн приложения:

image

Расскажите как вы это себе видите

  1. При наведении на задачу, помимо кнопки удаления должны быть уже реализованные кнопки "Like" и "Important"
  2. В шапке приложения должен быть счетчик количества задач
  3. Анимации и прочие динамические эффекты - на усмотрение
  4. При нажатии на кнопку бургер меню в хедере, должно появляться меню с методами фильтрации задач
  5. В остальном все должно соответствовать дизайну
@ikorol3107
Copy link
Contributor

@TFK70 есть вопросы по таске:

  1. у меня в кнопке "+ New task" есть захардкоженые значения, их можно оставить?:
<Button
        width='fit-content'
        height='80px'
        position='relative'
        top='-45px'
        left='170px'
        fontSize='large'
        fontWeight='bold'
        color='white'
        backgroundColor='deepPurple'
        borderRadius='gigant'
        boxShadow='violet'
        onClick={() => {
          setInputStatus('visible')
          if (inputStatus === 'visible' && newNote !== '') {
            setNotes(addItem(notes, newNote))
            setNewNote('')
            setInputStatus('invisible')
          } else if (inputStatus === 'visible') {
            setInputStatus('invisible')
          }
        }}
      >
        <Space count={5} />
        {intl.formatMessage(messages.newTask)}
        <Space count={5} />
</Button>
  1. Есть ли какой-то функционал у кнопки меню в Хэдере?

@TFK70
Copy link
Author

TFK70 commented May 7, 2021

@TFK70 есть вопросы по таске:

  1. у меня в кнопке "+ New task" есть захардкоженые значения, их можно оставить?:
<Button
        width='fit-content'
        height='80px'
        position='relative'
        top='-45px'
        left='170px'
        fontSize='large'
        fontWeight='bold'
        color='white'
        backgroundColor='deepPurple'
        borderRadius='gigant'
        boxShadow='violet'
        onClick={() => {
          setInputStatus('visible')
          if (inputStatus === 'visible' && newNote !== '') {
            setNotes(addItem(notes, newNote))
            setNewNote('')
            setInputStatus('invisible')
          } else if (inputStatus === 'visible') {
            setInputStatus('invisible')
          }
        }}
      >
        <Space count={5} />
        {intl.formatMessage(messages.newTask)}
        <Space count={5} />
</Button>
  1. Есть ли какой-то функционал у кнопки меню в Хэдере?
  1. Не понял сути вопроса. Во-первых тут захардкоженных значений вроде как бы нет, просто компонент реализован иначе, чем другие. Во-вторых, конечно, делать конкретно так - не стоит, но местами подобное может быть необходимостью, и если без этого ничего не работает - то к чему это тогда убирать?
  2. Да, там должно быть выдвижное меню, где можно выбрать метод фильтрации задач, таску обновил. Его дизайн - на усмотрение. Это может быть как drawer, так и обычная модалка

@ikorol3107
Copy link
Contributor

@TFK70 в компонент PostControls из стора передаю значение const [setSearchValue] = useSearchValue() . Применяется здесь:

...
<Input
    width='100%'
    placeholder={intl.formatMessage(messages.search)}
    onChange={event => setSearchValue(event.target.value)}
/>
...

значение searchValue из стора не передаю, т.к. оно в этом компоненте не используется, НО без этого значения перестал работать этот фильтр по поиску.
Описание ошибки:
Uncaught TypeError: setSearchValue is not a function
at onChange (PostControls.tsx:63)
at HTMLUnknownElement.callCallback (react-dom.development.js:3945)

Не могу самостоятельно понять, что не так.

@TFK70
Copy link
Author

TFK70 commented May 12, 2021

@TFK70 в компонент PostControls из стора передаю значение const [setSearchValue] = useSearchValue() . Применяется здесь:

...
<Input
    width='100%'
    placeholder={intl.formatMessage(messages.search)}
    onChange={event => setSearchValue(event.target.value)}
/>
...

значение searchValue из стора не передаю, т.к. оно в этом компоненте не используется, НО без этого значения перестал работать этот фильтр по поиску.
Описание ошибки:
Uncaught TypeError: setSearchValue is not a function
at onChange (PostControls.tsx:63)
at HTMLUnknownElement.callCallback (react-dom.development.js:3945)

Не могу самостоятельно понять, что не так.

useState всегда возвращает массив из 2 значений: переменную состояния и функцию. У тебя оттуда только переменная и берётся
То, что в её названии есть префикс set не делает её функцией

Чтобы пропустить первое значение, возвращаемое useState'ом можно сделать так:

const [,setSearchValue] = useSearchValue()

ikorol3107 added a commit that referenced this issue May 12, 2021
affects: @my-diary/app, @components/main-page-header, @components/post-controls, @practice/my-diary,
@store/show-drawer, @ui/button, @ui/drawer, @ui/input, @ui/list, @ui/theme

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 13, 2021
affects: @my-diary/app, @components/main-page-header, @components/post-controls, @practice/my-diary,
@ui/drawer, @ui/input, @ui/list, @ui/theme

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 14, 2021
affects: @components/post-controls, @practice/my-diary, @ui/drawer

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 14, 2021
affects: @my-diary/app, @components/post-controls, @practice/my-diary, @ui/drawer, @ui/layout

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 14, 2021
affects: @my-diary/app, @components/post-controls, @practice/my-diary, @ui/button, @ui/drawer

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 15, 2021
affects: @my-diary/app, @components/form-add-post, @components/main-page-header,
@components/post-controls, @practice/my-diary, @ui/button, @ui/drawer, @ui/input, @ui/list,
@ui/text, @ui/theme

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 16, 2021
…states

affects: @my-diary/app, @components/post-list, @practice/my-diary, @ui/button, @ui/drawer, @ui/list,
@ui/theme

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 17, 2021
…ata file

affects: @my-diary/app, @fragments/form-add-post, @fragments/main-page-header,
@fragments/post-controls, @fragments/post-list, @practice/my-diary, @ui/list

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 17, 2021
affects: @fragments/form-add-post, @practice/my-diary, @ui/list

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 18, 2021
affects: @my-diary/app, @fragments/main-page-header, @practice/my-diary, @ui/background, @ui/icons

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 18, 2021
affects: @my-diary/app, @fragments/diary, @practice/my-diary

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 18, 2021
affects: @fragments/form-add-post, @practice/my-diary, @ui/button

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 19, 2021
affects: @my-diary/app, @fragments/diary, @fragments/form-add-post, @practice/my-diary,
@ui/background, @ui/button, @ui/list, @ui/theme

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 19, 2021
affects: @fragments/form-add-post, @practice/my-diary, @ui/button

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 19, 2021
affects: @fragments/form-add-post, @practice/my-diary

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 19, 2021
affects: @fragments/form-add-post, @practice/my-diary

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 20, 2021
affects: @fragments/diary, @fragments/form-add-post, @practice/my-diary, @ui/condition

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 20, 2021
affects: @fragments/diary, @fragments/form-add-post, @practice/my-diary, @ui/background, @ui/button,
@ui/condition

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 22, 2021
affects: @practice/my-diary, @ui/button, @ui/icons

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 22, 2021
affects: @fragments/form-add-post, @fragments/main-page-header, @fragments/post-controls,
@practice/my-diary, @ui/button, @ui/filter, @ui/input, @ui/layout, @ui/list, @ui/theme

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 26, 2021
affects: @fragments/form-add-post, @fragments/post-controls, @practice/my-diary, @ui/button,
@ui/filter, @ui/input

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 26, 2021
affects: @my-diary/app, @fragments/form-add-post, @fragments/main-page-header,
@fragments/post-controls, @practice/my-diary, @ui/filter, @ui/input, @ui/text, @ui/theme

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 27, 2021
affects: @fragments/post-controls, @practice/my-diary, @ui/filter, @ui/input, @ui/text, @ui/theme

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 27, 2021
affects: @practice/my-diary, @ui/input, @ui/text, @ui/theme

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 27, 2021
affects: @practice/my-diary, @ui/button, @ui/condition, @ui/drawer, @ui/input

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 28, 2021
affects: @practice/my-diary, @ui/button, @ui/condition, @ui/input, @ui/theme

ISSUES CLOSED: #77
ikorol3107 added a commit that referenced this issue May 31, 2021
affects: @fragments/diary, @fragments/form-add-post, @practice/my-diary, @ui/background, @ui/button,
@ui/condition, @ui/filter, @ui/input, @ui/list, @ui/theme

ISSUES CLOSED: #77
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants