Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: input component #12

Closed
wants to merge 2 commits into from
Closed

Conversation

kirill-ivanovvv
Copy link
Member

@kirill-ivanovvv kirill-ivanovvv commented May 8, 2024

closes #11

feat: input component
@kirill-ivanovvv kirill-ivanovvv requested a review from Nelfimov May 8, 2024 18:11
@kirill-ivanovvv
Copy link
Member Author

хочу отрендерить компонент Input на странице, убедиться что все работает.

в инпут не попадают свойства theme

  ../.yarn/__virtual__/@ui-input-virtual-6615bb79b7/1/ui/input/src/input.styles.ts (31:26) @ input
  TypeError: Cannot read properties of undefined (reading 'input')
    at primaryColorsStyles (../../../../.yarn/__virtual__/@ui-input-virtual-6615bb79b7/1/ui/input/src/input.styles.ts:38:37)
  29 | export const primaryColorsStyles = ({theme}) => ({
  30 | 	...createAppearanceStyles({
> 31 | 		fontColor: theme.colors.input.font,
     | 		                      ^
  32 | 		backgroundColor: theme.colors.input.background,
  33 | 		borderColor: theme.colors.input.border,
  34 | 	})(),

для теста добавил в объект theme эти свойства, в theme.provider выходит такой лог,:
тоесть это свойство в объекте есть.

{ font: 'rgb(128, 127, 127)' }

мне кажется, что ошибка где-то тут, не понимаю как передается объект theme в компонент Input

const Page = () => {
	return (
		<ThemeProvider>
			<Input
				placeholder='input-placeholder'
			/>
		</ThemeProvider>
	)

}

Copy link
Member

@Nelfimov Nelfimov left a comment

Choose a reason for hiding this comment

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

  1. Пользуйся yarn format для читаемости кода
  2. У тебя потерялся контекст - @emotion/react как зависимость была только у theme, а @ui/input ее не видел. Обрати внимание на мой коммит

@Nelfimov
Copy link
Member

Nelfimov commented May 9, 2024

Ну и в добавок - нейминг веток. Имена собственные не допускаются.

@kirill-ivanovvv
Copy link
Member Author

Ну и в добавок - нейминг веток. Имена собственные не допускаются.

image

@Nelfimov
Copy link
Member

Nelfimov commented May 9, 2024

Ну и в добавок - нейминг веток. Имена собственные не допускаются.

image

ок, в этот раз прощаю

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.

сборка части ui компонентов
2 participants