Skip to content

Feature/time picker - #41

Open
TxMk2 wants to merge 18 commits into
developfrom
feature/time-picker
Open

Feature/time picker#41
TxMk2 wants to merge 18 commits into
developfrom
feature/time-picker

Conversation

@TxMk2

@TxMk2 TxMk2 commented Sep 24, 2025

Copy link
Copy Markdown
Contributor

概要

  • time pickerの実装
  • clock time pickerの実装

詳細

動作確認

  • time picker
    export-1771744622764

  • clock time picker
    export-1771745030881

確認項目

  • 動作確認を実施している
  • issueはPRページ右下のDevelopmentからissueが紐づいている

@TxMk2
TxMk2 requested a review from kuu13580 September 24, 2025 15:55
@TxMk2 TxMk2 self-assigned this Sep 24, 2025
@TxMk2 TxMk2 added the enhancement New feature or request label Sep 24, 2025
@github-actions

github-actions Bot commented Feb 1, 2026

Copy link
Copy Markdown

Visit the preview URL for this PR (updated for commit db31777):

https://tabi-share-8ef6b--pr41-feature-time-picker-4v02zcn5.web.app

(expires Mon, 06 Apr 2026 16:52:27 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 9f2a87ede127df7673322845e34cf22c1372d720

@TxMk2
TxMk2 force-pushed the feature/time-picker branch from 77c56b3 to c20b399 Compare February 22, 2026 07:05
@TxMk2 TxMk2 linked an issue Feb 22, 2026 that may be closed by this pull request
@TxMk2
TxMk2 marked this pull request as ready for review February 22, 2026 07:32
@TxMk2

TxMk2 commented Feb 22, 2026

Copy link
Copy Markdown
Contributor Author

@kuu13580
レビューよろです

@kuu13580

Copy link
Copy Markdown
Contributor

@TxMk2
めっちゃいい感じなんやけど、モバイルでちゃんと動作していなささそう

  • iPad Pro
  • Android (Google Pixel 10)

@TxMk2

TxMk2 commented Mar 25, 2026

Copy link
Copy Markdown
Contributor Author

@TxMk2 めっちゃいい感じなんやけど、モバイルでちゃんと動作していなささそう

* iPad Pro

* Android (Google Pixel 10)

まじや、かなり反応が悪い。。。。

@TxMk2

TxMk2 commented Mar 25, 2026

Copy link
Copy Markdown
Contributor Author

@kuu13580
レビューよろやで

const CLOCK_CENTER = CLOCK_SIZE / 2;
const NUMBER_RADIUS = CLOCK_SIZE / 2 - 20;

function ClockTimePicker() {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@TxMk2
実装あざます!動いた
けど、このコンポーネント外部となんのやり取りもしてないから値を取れない...
通常のinput要素と同様のinterfaceを継承して値をonChange(内部実装で隠蔽)で受け取れるようにしてくれい

type ClockTimePickerProps = Omit<React.ComponentProps<'input'>, 'type'>;                                          
  function ClockTimePicker({ value, onChange, ...props }: ClockTimePickerProps)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

カレンダーのコンポーネントを作成

2 participants