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(calendar, date-picker, date-range-picker): 今日の日付を目立つようにする & キャンセル・適用ボタンの追加 React #1364

Merged
merged 20 commits into from
Dec 26, 2024

Conversation

k-tada
Copy link
Contributor

@k-tada k-tada commented Sep 9, 2024

タスク

resolve: #1347

Copy link

changeset-bot bot commented Sep 9, 2024

🦋 Changeset detected

Latest commit: 160354f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages
Name Type
@wizleap-inc/wiz-ui-react Minor
@wizleap-inc/wiz-ui-constants Minor
@wizleap-inc/wiz-ui-styles Minor
@wizleap-inc/wiz-ui-next Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

vercel bot commented Sep 9, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
wiz-ui-next ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 26, 2024 4:57am
wiz-ui-react ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 26, 2024 4:57am

Copy link
Contributor

github-actions bot commented Sep 9, 2024

Size Change: +1.59 kB (+0.47%)

Total Size: 342 kB

Filename Size Change
./packages/wiz-ui-next/dist/style.css 11.9 kB +10 B (+0.08%)
./packages/wiz-ui-next/dist/wiz-ui.es.js 87.7 kB +502 B (+0.58%)
./packages/wiz-ui-next/dist/wiz-ui.umd.js 77.3 kB +395 B (+0.51%)
./packages/wiz-ui-react/dist/style.css 12.1 kB +14 B (+0.12%)
./packages/wiz-ui-react/dist/wiz-ui.es.js 81.5 kB +360 B (+0.44%)
./packages/wiz-ui-react/dist/wiz-ui.umd.js 71.7 kB +310 B (+0.43%)

compressed-size-action

Copy link

reg-suit bot commented Sep 9, 2024

reg-suit detected visual differences.

Check this report, and review them.

🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴🔴
⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪⚪

🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵🔵

What do the circles mean? The number of circles represent the number of changed images.
🔴 : Changed items, ⚪ : New items, ⚫ : Deleted items, and 🔵 Passed items

How can I change the check status? If reviewers approve this PR, the reg context status will be green automatically.

@k-tada k-tada changed the title Feat(calendar): 今日の日付を目立つようにする Feat(calendar): 今日の日付を目立つようにする & キャンセル・適用ボタンの追加 Sep 10, 2024
@k-tada k-tada changed the title Feat(calendar): 今日の日付を目立つようにする & キャンセル・適用ボタンの追加 Feat(calendar, date-picker, date-range-picker): 今日の日付を目立つようにする & キャンセル・適用ボタンの追加 Sep 10, 2024
@k-tada k-tada changed the title Feat(calendar, date-picker, date-range-picker): 今日の日付を目立つようにする & キャンセル・適用ボタンの追加 Feat(calendar, date-picker, date-range-picker): 今日の日付を目立つようにする & キャンセル・適用ボタンの追加 React Sep 10, 2024
Copy link
Collaborator

@ichi-h ichi-h left a comment

Choose a reason for hiding this comment

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

コードは問題ないと思うのですが、挙動面で2つお願いいたします!
(vueの方はまだ見れていないのですが、同じような挙動になっていないかご確認をお願いいたします 🙏 )


(date-range-picker) popupを欄外クリックで閉じたときに入力項目がもとに戻らない

  • 再現手順
    • 何らかの日付が選択されている状態でpopupを開く
    • 日付を適当に選択する(適用ボタンは押さない)
    • popupの外側をクリックして閉じる
    • 再度popupを開くと、現在選択されている日付がcalendar上で選択されていない(過去に編集した日付が選択されたままになっている)
  • 想定する挙動
    • popupの外側をクリックして閉じたときに、入力項目がもとに戻る(キャンセルボタンを押したときと同じ挙動)

(date-range-picker) セレクターのキャンセルが効かない

  • 再現手順
    • popupを開き、適当にセレクターを選択する
    • キャンセルボタンを押す or popupの外側をクリックして閉じる
    • 再度popupを開くと、セレクターが変更されたままになっている
  • 想定する挙動
    • キャンセルボタンを押す or popupの外側をクリックして閉じたときに、セレクターがもとに戻る

@ichi-h ichi-h removed the 🚧 wip Work In Progress label Sep 24, 2024
@k-tada
Copy link
Contributor Author

k-tada commented Nov 12, 2024

@ichi-h
修正しました!

@k-tada k-tada requested a review from ichi-h November 12, 2024 09:33
@ichi-h
Copy link
Collaborator

ichi-h commented Dec 23, 2024

  • なんか選択する
  • 選択を消す
  • ポップアップ開く
  • 過去選択していたものが残ったまま
  • 消したい

k-tada and others added 3 commits December 24, 2024 18:41
Feat(calendar, date-picker, date-range-picker): 今日の日付を目立つようにする & キャンセル・適用ボタンの追加 Vue
Copy link
Collaborator

@ichi-h ichi-h left a comment

Choose a reason for hiding this comment

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

changesetだけいじっておきます!
ありがとうございます!! 🙏

@ichi-h ichi-h merged commit 26c0d64 into main Dec 26, 2024
8 of 9 checks passed
@ichi-h ichi-h deleted the feat/calendar-today-react branch December 26, 2024 05:00
@github-actions github-actions bot mentioned this pull request Dec 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants