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

TextField : Label Typography 적용되도록 수정, Width Property 추가 #41

Merged
merged 1 commit into from
Aug 22, 2023

Conversation

bce0404
Copy link
Collaborator

@bce0404 bce0404 commented Aug 21, 2023

image

  1. TextField의 "Label" Property에 의해 표시되는 텍스트에 Typography가 적용되지 않고 있었음.
    -> Typography 적용되도록 div -> Typography로 변경

  2. mui Box를 사용하던 부분을 MoaStack으로 변경.

  3. Width Property 추가 (이게 메인입니다)

작업하다가 필요하여 수정한 부분들을 묶어서 PR올립니다.

@bce0404 bce0404 added the enhancement New feature or request label Aug 21, 2023
@bce0404 bce0404 requested review from HY0118 and kh1012 August 21, 2023 10:55
Copy link
Collaborator

@HY0118 HY0118 left a comment

Choose a reason for hiding this comment

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

approve~!

Copy link
Collaborator

@kh1012 kh1012 left a comment

Choose a reason for hiding this comment

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

approval!

* width="10ex"
* width="10px"
*/
width?: string,
Copy link
Collaborator

Choose a reason for hiding this comment

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

점차 진화하는 tsdoc

@@ -21,6 +22,9 @@ function Demo() {
<TextField placeholder={'placeholder'} />
<br/><br/>
<TextField placeholder={'value'} value="valueTest"/>
<MoaStack>
<TextField placeholder={'value'} title="label" titlePosition="label" width="100%" value="width Test"/>
Copy link
Collaborator

Choose a reason for hiding this comment

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

이건 궁금증!
placeholder={'value'}
placeholder='value'
이게 혹시 차이가 있나요?

Copy link
Collaborator Author

@bce0404 bce0404 Aug 22, 2023

Choose a reason for hiding this comment

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

{ }는 값으로 주어진 변수를 참조하고, '' 또는 ""는 상수를 선언합니다.
변수에서 상수를 참조하는 것은 결국 상수인 셈이라 차이가 없을 것 같네요 ㅎ

@HY0118 HY0118 merged commit d0227c2 into dev Aug 22, 2023
@bce0404 bce0404 deleted the feature/TextField_Width_Typography branch August 28, 2023 02:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants