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: 月選択UI (MonthPicker) を追加したい #5030

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

zoshigayan
Copy link
Contributor

@zoshigayan zoshigayan commented Oct 18, 2024

関連URL

  • 🍐

概要

  • 任意の月 (年月) を選択するUIを追加したいです
  • input[type=month] をラップしたものです

変更内容

  • MonthPicker を追加しました
  • 実装がほぼ TimePicker と同じになるので、同一ディレクトリにまとめて型定義やスタイルを共通化しています

確認方法

  • storybookの確認のみでOKだと思います

@zoshigayan zoshigayan self-assigned this Oct 18, 2024
Copy link

pkg-pr-new bot commented Oct 18, 2024

Open in Stackblitz

pnpm add https://pkg.pr.new/kufu/smarthr-ui@5030

commit: 3ef0f2c

@zoshigayan zoshigayan marked this pull request as ready for review October 18, 2024 08:36
@zoshigayan zoshigayan requested a review from a team as a code owner October 18, 2024 08:36
@zoshigayan zoshigayan requested review from Qs-F and masa0527 and removed request for a team October 18, 2024 08:36
Copy link
Contributor

@Qs-F Qs-F left a comment

Choose a reason for hiding this comment

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

ありがとうございます〜!挙動は良さそうでした!
少し気になったところを書いておきます!

  • Pickerだと少し広い名前に感じてしまいました。DateTimePicker とかでまとめてもいいかも
  • DatePickerもPickerに移動させてもいいかもと思いました!
    • 既にSlackで出てた結論でした!上記2つは無視で大丈夫です
  • input[type=month] がSafariやFirefoxで対応していないようでただのテキストになってしまう雰囲気を感じたのですが、それは問題なさそうですか?

@Qs-F
Copy link
Contributor

Qs-F commented Oct 18, 2024

@uknmr 新規コンポーネント追加するときって新しいStorybookの形式にしてもらったほうがいいですかね?(とはいえドキュメントが充実しているわけではない気もするので若干やってもらう心苦しさはありつつ…)

@s-sasaki-0529
Copy link
Contributor

あとこれも全然運用プロセスに乗せてないんですが、RSCに対応してるかどうかの定義追加もお願いしたいです。
コード見る限りはRSCでいけそうなので、以下に追加してテストが通ればOKです!

/**
* サーバーコンポーネントとして利用できるコンポーネント一覧(アルファベット順)
*/
const SERVER_COMPONENTS = [
'AnchorButton',
'AppNaviAnchor',
'AppNaviButton',
'AppNaviCustomTag',
'Badge',
'Balloon',
'Chip',
'HeaderLink',
'Icon',
'Loader',
'RangeSeparator',
'ResponseMessage',
'SmartHRLogo',
'SpreadsheetTable',
'SpreadsheetTableCorner',
'StatusLabel',
'Table',
'Td',
'Text',
'TextLink',
'Th',
'TimePicker',
'UnstyledButton',
'UpwardLink',
'VisuallyHiddenText',
]

@zoshigayan
Copy link
Contributor Author

@Qs-F #5030 (review)

input[type=month] がSafariやFirefoxで対応していないようでただのテキストになってしまう雰囲気を感じたのですが、それは問題なさそうですか?

あーすみません、ここ確認できてませんでした…!
いちおう SmartHRの動作環境 としているブラウザ (Chrome / Edge / iOS Safari / Android Chrome) が対応していそうなので社内プロダクトとしての利用に限って見れば大丈夫だと思います 💭
SmartHR UIの対応環境って特に明記されていないと思うので、上記で大丈夫なのかどうか…については皆さんにご意見いただきたいです 🙏

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.

見ました!RSC検証の対応もありがとうございます!

個人的にはですが

  • Storybook のリライト作業はチームミッションとしての取り組みなので、少なくとも期中はコアチームでやりきるで良さそう
  • ブラウザサポートも SmartHR の範囲内では扱えてるし、Firefox などでもただのテキストボックスになるだけで辛うじて使えないこともないので良さそう

なお気持ちです。

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