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!: Dialog の位置指定を消し、スタイリングを見直し #4848

Merged
merged 5 commits into from
Sep 10, 2024

Conversation

uknmr
Copy link
Collaborator

@uknmr uknmr commented Aug 14, 2024

Related URL

https://smarthr.atlassian.net/browse/SHRUI-702

Overview

ActionDialog や MessageDialog、FormDialog から絶対位置指定のための props(top / right / bottom / left)を消しました。

また、今後の構造改修やモバイル対応を見据えてスタイリング周りの記述を見直しました。

What I did

  • 位置指定を消すとともに、幅や高さの算出をやめ CSS Calculation でスタイリングしました
  • DialogHeader や DialogBody を共通化しました
    • DialogFooter も共通化したかったんですが、時間切れのため別でやります

Capture

@uknmr uknmr requested a review from a team as a code owner August 14, 2024 06:06
@uknmr uknmr requested review from moshisora and s-sasaki-0529 and removed request for a team August 14, 2024 06:06
@uknmr uknmr marked this pull request as draft August 14, 2024 07:00
@uknmr uknmr force-pushed the remove-Dialog-position branch from f4d5444 to 66d77be Compare August 14, 2024 07:05
@uknmr uknmr force-pushed the remove-Dialog-position branch from 66d77be to c81b121 Compare August 14, 2024 07:07
@uknmr uknmr marked this pull request as ready for review August 14, 2024 07:07
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.

これってプロダクトからは全然使用されていないって確認できてるんでしたっけ・・・?

Copy link
Contributor

@oti oti left a comment

Choose a reason for hiding this comment

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

LGTM!

@uknmr uknmr merged commit 25d83ec into master Sep 10, 2024
8 checks passed
@uknmr uknmr deleted the remove-Dialog-position branch September 10, 2024 00:15
@uknmr
Copy link
Collaborator Author

uknmr commented Sep 10, 2024

@s-sasaki-0529 org:kufu Dialog top などの条件で検索し、利用が少ないことを確認済みです。また、外からスタイリングを上書けることも確認できたため消す判断をしました。

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.

3 participants