Skip to content

feat: 「いま」の変換で現在時刻の入力に対応#298

Merged
ensan-hcl merged 1 commit intoazooKey:mainfrom
patt812:feat/add-time-conversion
Mar 3, 2026
Merged

feat: 「いま」の変換で現在時刻の入力に対応#298
ensan-hcl merged 1 commit intoazooKey:mainfrom
patt812:feat/add-time-conversion

Conversation

@patt812
Copy link
Contributor

@patt812 patt812 commented Mar 3, 2026

Summary

Google 日本語入力での「いま」変換

image

azooKey での「いま」変換

image

変更内容

  • SegmentsManager.swiftdynamicShortcuts に「イマ」をトリガーとする DateTemplateLiteral エントリを3件追加

Test plan

  • swift test --package-path Coreが全部パスすることを確認
  • SwiftLintで違反なし
  • ./install.sh でビルド&インストール後、入力ソースにazooKeyを追加して動作確認
  • 「いま」と入力し、3つの時刻フォーマットが変換候補に表示されることを確認

@ensan-hcl
Copy link
Member

Google日本語入力とのCompatibilityは特に意図していないので、これを導入するかはちょっと考え中です。

@ensan-hcl
Copy link
Member

以下の議論のもとで取り込むことにします。

  • 前提: azooKey (macOS)ではGoogle日本語入力との機能アラインメントを考えているわけではないので、Google日本語入力に存在する機能だからといって、必ずしも取り込む必要があるとは考えていない
  • 今回の機能追加に関しては、以下の理由で取り込んで良さそう
    • 特に強い副作用はない
      • 強いて言うなら「居間」とかの変換が場合によってはちょっとめんどくさくなる
    • 実装量も軽微(すでに実装済みの基盤に乗っているため)
    • 頻繁にリクエストがある

@ensan-hcl ensan-hcl self-requested a review March 3, 2026 16:09
@ensan-hcl ensan-hcl added the enhancement New feature or request label Mar 3, 2026
Copy link
Member

@ensan-hcl ensan-hcl left a comment

Choose a reason for hiding this comment

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

LGTMです!

@ensan-hcl ensan-hcl merged commit 7cd4990 into azooKey:main Mar 3, 2026
4 checks passed
@patt812
Copy link
Contributor Author

patt812 commented Mar 4, 2026

マージありがとうございます!

補足ですが、今回のPRはGoogle日本語入力との互換性を目的としたものではなく、個人的にazooKeyを使っていて時刻を入力できると嬉しいと思ったことがきっかけです。
フォーマットについては、#276 の日付変換PRの様式に倣いGoogle日本語入力を参考にしましたが、今回はmacOSの入力は該当なく、まずは3パターンで良さそうと考えこの形に至りましたmm

互換性については今後気をつけます。ありがとうございます!mm

@patt812 patt812 deleted the feat/add-time-conversion branch March 4, 2026 15:52
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.

[feature] 「いま」の入力の変換候補にタイムスタンプを提案

2 participants