Borodutch/fondu
Watch
1
0
Code Issues Pull requests Projects Releases Wiki Activity Actions

Create new slider and fix mobile version #44

Merged
mormo669 merged 8 commits from new-slider into v2.0 2021-09-16 22:53:44 +00:00
Conversation 7 Commits 8 Files changed 12 +112 -33
mormo669 commented 2021-08-24 06:10:40 +00:00 (Migrated from github.com)
Copy link
No description provided.
backmeupplz (Migrated from github.com) requested changes 2021-08-25 01:16:10 +00:00
src/components/Button.tsx
@ -10,7 +10,6 @@ const baseButtonStyle = classnames(
'focus:outline-none',
backmeupplz (Migrated from github.com) commented 2021-08-25 01:10:08 +00:00
Copy link

Не оставляй закомментированный код в пулл реквестах

Не оставляй закомментированный код в пулл реквестах
src/components/Header.tsx
@ -1,5 +1,4 @@
import { FC } from 'react'
backmeupplz (Migrated from github.com) commented 2021-08-25 01:10:22 +00:00
Copy link

Удали закомментированный код

Удали закомментированный код
src/components/Header.tsx
@ -9,9 +8,10 @@ import { classnames } from 'classnames/tailwind'
const headerStyle = classnames(
'mb-10',
backmeupplz (Migrated from github.com) commented 2021-08-25 01:10:26 +00:00
Copy link

Удали закомментированный код

Удали закомментированный код
src/components/Header.tsx
@ -22,7 +22,6 @@ const Header: FC = () => {
</LogoText>
backmeupplz (Migrated from github.com) commented 2021-08-25 01:10:29 +00:00
Copy link

Удали закомментированный код

Удали закомментированный код
src/components/Slider.tsx
@ -0,0 +1,79 @@
import { observer } from 'mobx-react-lite'
backmeupplz (Migrated from github.com) commented 2021-08-25 01:11:41 +00:00
Copy link

Здесь не () нужны, а {}

Здесь не `()` нужны, а `{}`
backmeupplz (Migrated from github.com) commented 2021-08-25 01:11:48 +00:00
Copy link

Здесь не нужны {}

Здесь не нужны `{}`
backmeupplz (Migrated from github.com) commented 2021-08-25 01:12:42 +00:00
Copy link

Если внутри ничего нет, сделай самозакрывающийся тег типа <tag />

Если внутри ничего нет, сделай самозакрывающийся тег типа `<tag />`
backmeupplz (Migrated from github.com) commented 2021-08-25 01:12:53 +00:00
Copy link

Тут нужны {}

Тут нужны `{}`
backmeupplz (Migrated from github.com) commented 2021-08-25 01:13:04 +00:00
Copy link

Тут нужен самозакрывающийся тег

Тут нужен самозакрывающийся тег
backmeupplz (Migrated from github.com) commented 2021-08-25 01:13:11 +00:00
Copy link

Здесь не нужны {}

Здесь не нужны `{}`
src/components/Wallet/index.tsx
backmeupplz (Migrated from github.com) commented 2021-08-25 01:12:21 +00:00
Copy link

Классы только через classnames

Классы только через classnames
src/views/ERC20TokenSettings.tsx
backmeupplz (Migrated from github.com) commented 2021-08-25 01:12:07 +00:00
Copy link

Классы только через classnames

Классы только через classnames
src/views/ERC721TokenSettings.tsx
backmeupplz (Migrated from github.com) commented 2021-08-25 01:11:08 +00:00
Copy link

Классы — только через classnames

Классы — только через `classnames`
mormo669 (Migrated from github.com) reviewed 2021-08-28 05:27:55 +00:00
src/components/Wallet/index.tsx
mormo669 (Migrated from github.com) commented 2021-08-28 05:27:55 +00:00
Copy link

при classnames выдаёт ошибку и предлагает замену на className

при classnames выдаёт ошибку и предлагает замену на className
backmeupplz (Migrated from github.com) requested changes 2021-08-29 21:43:22 +00:00
src/components/Button.tsx
@ -64,3 +67,3 @@
}
)}
>
{!!icon && <img src={icon} alt={icon} />}
backmeupplz (Migrated from github.com) commented 2021-08-29 21:42:05 +00:00
Copy link

Тебе не нужно экспортить buttonSize. Если ты везде оборачиваешь кнопку в контейнер с buttonSize, так просто оберни его в этом компоненте. Если нужно где-то оборачивать, а где-то нет — добавь props в Button компонент и настраивай стили соответственно.

Тебе не нужно экспортить `buttonSize`. Если ты везде оборачиваешь кнопку в контейнер с `buttonSize`, так просто оберни его в этом компоненте. Если нужно где-то оборачивать, а где-то нет — добавь `props` в `Button` компонент и настраивай стили соответственно.
src/views/TokenType.tsx
backmeupplz (Migrated from github.com) commented 2021-08-29 21:40:48 +00:00
Copy link

Классы только через classnames

Классы только через `classnames`
backmeupplz (Migrated from github.com) requested changes 2021-09-05 23:37:39 +00:00
src/components/Button.tsx
@ -33,6 +32,7 @@ const selectedButtonStyle = classnames(
'bg-main',
backmeupplz (Migrated from github.com) commented 2021-09-05 23:37:31 +00:00
Copy link

Что такое buttonSize? Непонятно. Назови переменную понятнее.

Что такое `buttonSize`? Непонятно. Назови переменную понятнее.
src/components/Button.tsx
@ -50,18 +51,20 @@ const Button: FC<
title,
backmeupplz (Migrated from github.com) commented 2021-09-05 23:37:04 +00:00
Copy link

Какой-то непонятный хак, его быть не должно. Если хочешь — делай тут массив, а не соединяй строки

Какой-то непонятный хак, его быть не должно. Если хочешь — делай тут массив, а не соединяй строки
src/components/Slider.tsx
@ -0,0 +1,79 @@
import { observer } from 'mobx-react-lite'
backmeupplz (Migrated from github.com) commented 2021-09-05 23:35:18 +00:00
Copy link

Если у этого компонента не надо указывать props, то не надо лишний раз указывать тип FC

Если у этого компонента не надо указывать props, то не надо лишний раз указывать тип `FC`
src/components/Slider.tsx
@ -0,0 +49,4 @@
{!!intlKey ? <FormattedMessage id={intlKey} /> : <span>{title}</span>}
</button>
)
}
backmeupplz (Migrated from github.com) commented 2021-09-05 23:34:55 +00:00
Copy link

Добавляй между компонентами пустую строку

Добавляй между компонентами пустую строку
src/components/Wallet/index.tsx
backmeupplz (Migrated from github.com) commented 2021-09-05 23:34:03 +00:00
Copy link

Не пушь в PR закомментированный код

Не пушь в PR закомментированный код
src/views/ERC721TokenSettings.tsx
@ -64,6 +64,7 @@ const ERC721TokenSettingsView: FC = () => {
<Button
backmeupplz (Migrated from github.com) commented 2021-09-05 23:33:47 +00:00
Copy link

Непонятно, что такое size.

Непонятно, что такое `size`.
backmeupplz (Migrated from github.com) requested changes 2021-09-08 23:11:24 +00:00
src/components/Wallet/index.tsx
@ -44,6 +44,7 @@ const ContractWallet: FC = () => {
<Button
backmeupplz (Migrated from github.com) commented 2021-09-08 23:10:50 +00:00
Copy link

Можно просто сделать maxsize — будет автоматом true, не нужно писать прямо maxSize={true}

Можно просто сделать `maxsize` — будет автоматом true, не нужно писать прямо `maxSize={true}`
mormo669 (Migrated from github.com) reviewed 2021-09-11 07:41:08 +00:00
src/components/Slider.tsx
@ -0,0 +1,79 @@
import { observer } from 'mobx-react-lite'
mormo669 (Migrated from github.com) commented 2021-09-11 07:41:07 +00:00
Copy link

Done

Done
backmeupplz (Migrated from github.com) requested changes 2021-09-11 19:10:07 +00:00
src/components/Slider.tsx
@ -0,0 +1,79 @@
import { observer } from 'mobx-react-lite'
backmeupplz (Migrated from github.com) commented 2021-09-11 19:10:04 +00:00
Copy link

Включи у себя prettier и format on save

Включи у себя prettier и format on save
Sign in to join this conversation.
Reviewers
No reviewers
backmeupplz
Labels
Clear labels   
bug
Something isn't working

  
dependencies
Pull requests that update a dependency file

  
duplicate
This issue or pull request already exists

  
enhancement
New feature or request

  
good first issue
Good for newcomers

  
help wanted
Extra attention is needed

  
invalid
This doesn't seem right

  
question
Further information is requested

  
wontfix
This will not be worked on

No labels
bug
dependencies
duplicate
enhancement
good first issue
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
Borodutch/fondu!44
No description provided.