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

[DS-89] Add Button Default Height #131

Merged
merged 2 commits into from
Oct 6, 2023
Merged

[DS-89] Add Button Default Height #131

merged 2 commits into from
Oct 6, 2023

Conversation

LTakhyunKim
Copy link
Contributor

@LTakhyunKim LTakhyunKim commented Sep 19, 2023

DS-89

Description

  • Button component 의 기본 height 값을 지정합니다.
    (Toggle Button 은 기본 Button style 을 공유하므로 따로 수정하지 않아도 적용됩니다.)

@LTakhyunKim LTakhyunKim self-assigned this Sep 19, 2023
@github-actions
Copy link

github-actions bot commented Sep 19, 2023

@deminoth
Copy link
Member

고정 height이 생겼으니 minHeight은 지워도 되지 않을까요?

@LTakhyunKim
Copy link
Contributor Author

LTakhyunKim commented Sep 19, 2023

고정 height이 생겼으니 minHeight은 지워도 되지 않을까요?

만약 외부에서 min Height 보다 낮은 값을 지정할 경우를 대비해서라도 있는 편이 좋을 것 같습니다.

@deminoth
Copy link
Member

만약 외부에서 min Height 보다 낮은 값을 지정할 경우를 대비해서라도 있는 편이 좋을 것 같습니다.

이건 어떤 경우인지 잘 모르겠네요 🤔 height을 지정하게 되면, 부모가 더 낮은 높이를 가지더라도 영향을 받지 않습니다. 만약 버튼 자체에 sx 등으로 높이를 지정한다면, 분명한 목적이 있는 것이니 그대로 줄어드는게 맞을 것 같습니다.

@LTakhyunKim
Copy link
Contributor Author

이건 어떤 경우인지 잘 모르겠네요 🤔 height을 지정하게 되면, 부모가 더 낮은 높이를 가지더라도 영향을 받지 않습니다. 만약 버튼 자체에 sx 등으로 높이를 지정한다면, 분명한 목적이 있는 것이니 그대로 줄어드는게 맞을 것 같습니다.

지정 시에 분명한 목적이 있다는 거를 생각 못했네요. 👍
말씀하신 것처럼 min height 는 삭제 조치했습니다. d33458c

@deminoth
Copy link
Member

머지해주세용

@LTakhyunKim LTakhyunKim merged commit 3973462 into main Oct 6, 2023
@LTakhyunKim LTakhyunKim deleted the ds-89-button-height branch October 6, 2023 05:32
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