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

fix: DefinitionListItem が空の場合に高さが潰れる欠陥を修正 #4639

Merged
merged 1 commit into from
May 10, 2024

Conversation

uknmr
Copy link
Collaborator

@uknmr uknmr commented May 10, 2024

Related URL

Overview

  • tailwind 移行時の不具合がありました
  • min-h-shr- prefix が付いておらず、値が効いていませんでした
  • 1文字分である calc(1em * leading.NORMAL) となるように修正しました
  • 再発しないよう Story を追加しました

リリースを待たずに修正したい場合は、.smarthr-ui-DefinitionListItem-description というクラスに対して min-height: calc(1em * leading.NORMAL) と同等のスタイルを充ててください。

What I did

Capture

@uknmr uknmr requested a review from a team as a code owner May 10, 2024 00:13
@uknmr uknmr requested review from yt-ymmt and nabeliwo and removed request for a team May 10, 2024 00:13
Copy link
Contributor

@s-sasaki-0529 s-sasaki-0529 left a comment

Choose a reason for hiding this comment

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

VRT 差分確認して高さが保たれてること確認しましたー。

shr- プレフィックス付けるの忘れそうですねぇ。リンターでカバーできると良いんですが。

@uknmr
Copy link
Collaborator Author

uknmr commented May 10, 2024

theme[hoge] などと書いていると、手元のエディタで補完が効かなくなってしまう問題も原因の1つだったりします(それを治しに行けばいいんだけど)。

@uknmr uknmr merged commit fa267f1 into master May 10, 2024
8 checks passed
@uknmr uknmr deleted the fix-DefinitionList-blank-height branch May 10, 2024 02:43
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.

None yet

2 participants