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

fix(frontend): フォーカス/タブ移動に関する挙動を調整 #226

Merged
merged 1 commit into from
Jun 5, 2024

Conversation

taiyme
Copy link
Owner

@taiyme taiyme commented May 27, 2024

What

  • PhotoSwipeで正しくフォーカストラップできるように修正
  • PhotoSwipe展開中はPhotoSwipe以外のホットキーが実行されないように変更
    • この変更により、PhotoSwipe展開中に新しくダイアログ/ポップアップが展開できなくなります
  • MkMenu(popupMenu/contextMenu)でフォーカストラップされるように変更
    • この変更により、ホットキー長押しで無限にメニューが展開される問題が解消されます
  • MkSelectでmousedownしたときにフォーカスされない問題を修正
    • この修正により、MkSelect実行中にテキスト入力などができないようになります
  • MkNote/MkNoteDetailedでホットキー"s"が機能していないのを修正
    • ただし、機能を拡充したホットキー"v"に置き換えている (検索のホットキーと被るので)
    • クリップメニューを開くことができる ホットキー"c" を追加
  • MkNoteDetailedでホットキーが殆どの場合動作しない問題を修正
  • 一部のclickによるpopupMenuをmousedownに変更
  • hotkeyで+(;)キーが動作していない問題を修正
  • ページ全体のoutlineを調整
    ほか

Why

Additional info (optional)

Checklist

  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If possible) Add tests

@taiyme taiyme self-assigned this May 27, 2024
@taiyme taiyme changed the title fix(frontend): フォーカス/タブ移動に関する挙動を調整 Draft: fix(frontend): フォーカス/タブ移動に関する挙動を調整 May 27, 2024
@taiyme taiyme force-pushed the fix-focus-tabbing branch 3 times, most recently from 5668ddf to 08a0368 Compare May 29, 2024 10:44
@taiyme taiyme marked this pull request as ready for review May 29, 2024 10:44
@taiyme taiyme changed the title Draft: fix(frontend): フォーカス/タブ移動に関する挙動を調整 fix(frontend): フォーカス/タブ移動に関する挙動を調整 May 29, 2024
@taiyme taiyme force-pushed the fix-focus-tabbing branch 4 times, most recently from e1e4185 to 283365b Compare June 4, 2024 08:00
@taiyme taiyme merged commit e8c0306 into taiyme Jun 5, 2024
2 checks passed
@taiyme taiyme deleted the fix-focus-tabbing branch June 5, 2024 07:06
truecharts-admin referenced this pull request in truecharts/charts Jul 31, 2024
…54d by renovate (#24613)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [misskey/misskey](https://github.com/misskey-dev/misskey) | minor |
`2024.5.0` -> `2024.7.0` |

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>misskey-dev/misskey (misskey/misskey)</summary>

###
[`v2024.7.0`](https://github.com/misskey-dev/misskey/blob/HEAD/CHANGELOG.md#202470)

[Compare
Source](https://github.com/misskey-dev/misskey/compare/2024.5.0...2024.7.0)

##### Note

-
デッキUIの新着ノートをサウンドで通知する機能の追加(v2024.5.0)に伴い、以前から動作しなくなっていたクライアント設定内の「アンテナ受信」「チャンネル通知」サウンドを削除しました。
- Streaming APIにて入力が不正な場合にはそのメッセージを無視するようになりました。
[#&#8203;14251](https://github.com/misskey-dev/misskey/issues/14251)

##### General

- Feat: 通報を受けた際、または解決した際に、予め登録した宛先に通知を飛ばせるように(mail or webhook)
[#&#8203;13705](https://github.com/misskey-dev/misskey/issues/13705)
-   Feat: ユーザーのアイコン/バナーの変更可否をロールで設定可能に
    -   変更不可となっていても、設定済みのものを解除してデフォルト画像に戻すことは出来ます
- Feat: ユーザ作成時にSystemWebhookを送信可能に
[#&#8203;14281](https://github.com/misskey-dev/misskey/issues/14281)
- Feat: メディアサイレンスを実装
[#&#8203;13842](https://github.com/misskey-dev/misskey/issues/13842)
-
メディアサイレンスされたサーバーに所属するアカウントによるファイルはすべてセンシティブとして扱われ、カスタム絵文字が使用できないようになります。
-   Enhance: 管理画面でアーカイブにしたお知らせを表示・編集できるように
-   Fix: 配信停止したインスタンス一覧が見れなくなる問題を修正
-   Fix: Dockerコンテナの立ち上げ時に`pnpm`のインストールで固まることがある問題
-   Fix: デフォルトテーマに無効なテーマコードを入力するとUIが使用できなくなる問題を修正
-   翻訳の更新
-   依存関係の更新

##### Client

- Feat: ユーザーページから「このユーザーのノートを検索」できるように
([#&#8203;14128](https://github.com/misskey-dev/misskey/issues/14128))
- Feat: 検索ページはクエリを受け付けるようになりました
([#&#8203;14128](https://github.com/misskey-dev/misskey/issues/14128))
- Enhance: 検索ページのUI改善
([#&#8203;14128](https://github.com/misskey-dev/misskey/issues/14128))
-   Enhance: 内蔵APIドキュメントのデザイン・パフォーマンスを改善
-   Enhance: 非ログイン時に他サーバーに遷移するアクションを追加
-   Enhance: 非ログイン時のハイライトTLのデザインを改善
-   Enhance: フロントエンドのアクセシビリティ改善\
(Based on
[https://github.com/taiyme/misskey/pull/226](https://github.com/taiyme/misskey/pull/226))
-   Enhance: サーバー情報ページ・お問い合わせページを改善\
(Cherry-picked from
[https://github.com/taiyme/misskey/pull/238](https://github.com/taiyme/misskey/pull/238))
-   Enhance: AiScriptを0.19.0にアップデート
- Enhance: Allow negative delay for MFM animation elements (`tada`,
`jelly`, `twitch`, `shake`, `spin`, `jump`, `bounce`, `rainbow`)
-   Enhance: センシティブなメディアを開く際に確認ダイアログを出せるように
- Enhance: 検索(ノート/ユーザー)で `#`
から始まる文字列を入力すると、そのハッシュタグのノート/ユーザー一覧ページが表示できるように
-   Enhance: 検索(ノート/ユーザー)において、入力に空白が含まれている場合は照会を行わないように
- Enhance:
検索(ノート/ユーザー)において、照会を行うかどうか、ハッシュタグのノート/ユーザー一覧ページを表示するかどうかの確認ダイアログを出すように
- Enhance: 検索(ノート/ユーザー)で `@`
から始まる文字列(`@user@host`など)を入力すると、そのユーザーを照会できるように
-   Enhance: ドライブのファイル・フォルダをドラッグしなくても移動できるように\
(Cherry-picked from
nafu-at/misskey@b89c2af,
nafu-at/misskey@8a7d710)
-   Enhance: デッキのアンテナ・リスト選択画面からそれぞれを新規作成できるように
-   Enhance: ブラウザのコンテキストメニューを使用できるように
-   Enhance: 連合の「連合中」,「購読中」,「配信中」に対してブロックしているサーバー、配信停止しているサーバーを含めないように
-   Fix: `/about#federation` ページなどで各インスタンスのチャートが表示されなくなっていた問題を修正
- Fix: ユーザーページの追加情報のラベルを投稿者のサーバーの絵文字で表示する
([#&#8203;13968](https://github.com/misskey-dev/misskey/issues/13968))
-   Fix: リバーシの対局を正しく共有できないことがある問題を修正
-   Fix: コントロールパネルでベースロールのポリシーを編集してもUI上では変更が反映されない問題を修正
-   Fix: アンテナの編集画面のボタンに隙間を追加
-   Fix: テーマプレビューが見れない問題を修正
-   Fix: ショートカットキーが連打できる問題を修正\
(Cherry-picked from
[https://github.com/taiyme/misskey/pull/234](https://github.com/taiyme/misskey/pull/234))
- Fix:
MkSignin.vueのcredentialRequestからReactivityを削除(ProxyがPasskey認証処理に渡ることを避けるため)
-   Fix: 「アニメーション画像を再生しない」がオンのときでもサーバーのバナー画像・背景画像がアニメーションしてしまう問題を修正\
(Cherry-picked from
https://activitypub.software/TransFem-org/Sharkey/-/merge_requests/574)
-   Fix: Twitchの埋め込みが開けない問題を修正
-   Fix: 子メニューの高さがウィンドウからはみ出ることがある問題を修正
-   Fix: 個人宛てのダイアログ形式のお知らせが即時表示されない問題を修正
-   Fix: 一部の画像がセンシティブ指定されているときに画面に何も表示されないことがあるのを修正
-   Fix: リアクションしたユーザー一覧のユーザー名がはみ出る問題を修正\
(Cherry-picked from
[https://github.com/MisskeyIO/misskey/pull/672](https://github.com/MisskeyIO/misskey/pull/672))
-   Fix: `/share`ページにおいて絵文字ピッカーを開くことができない問題を修正
- Fix: deck uiの通知音が重なる問題
([#&#8203;14029](https://github.com/misskey-dev/misskey/issues/14029))
-   Fix: ダイレクト投稿の"削除して編集"において、宛先が保持されていなかった問題を修正
-   Fix: 投稿フォームへのURL貼り付けによる引用が下書きに保存されていなかった問題を修正
-   Fix: "削除して編集"や下書きにおいて、リアクションの受け入れ設定が保持/保存されていなかった問題を修正
- Fix: 照会に `#` から始まる文字列を入力してそのハッシュタグのページを表示する際、入力が `#`
のみの場合に「指定されたURLに該当するページはありませんでした。」が表示されてしまう問題を修正
- Fix: 照会に `@` から始まる文字列を入力してユーザーを照会する際、入力が `@`
のみの場合に「問題が発生しました」が表示されてしまう問題を修正
- Fix:
投稿フォームにノートのURLを貼り付けて"引用として添付"した場合、投稿文を空にすることによるRenote化が出来なかった問題を修正
-   Fix: フォロー中のユーザーに関する"TLに他の人への返信を含める"の設定が分かりづらい問題を修正
-   Fix: タイムラインページを開いた時、`TLに他の人への返信を含める`がオフのときに`ファイル付きのみ`をオンにできない問題を修正
- Fix: deck
uiでタイムラインを切り替えた際にTLの設定項目が更新されず、`TLに他の人への返信を含める`のトグルが表示されない問題を修正
-   Fix: ウィジェットのタイムライン選択欄に無効化されたタイムラインが表示される問題を修正
-   Fix: サウンドにドライブの音声を使用している際にドライブの音声が再生できなくなると設定が変更できなくなる問題を修正

##### Server

- Feat: レートリミット制限に引っかかったときに`Retry-After`ヘッダーを返すように
([#&#8203;13949](https://github.com/misskey-dev/misskey/issues/13949))
-   Enhance: エンドポイント`clips/update`の必須項目を`clipId`のみに
-   Enhance: エンドポイント`admin/roles/update`の必須項目を`roleId`のみに
-   Enhance: エンドポイント`pages/update`の必須項目を`pageId`のみに
-   Enhance: エンドポイント`gallery/posts/update`の必須項目を`postId`のみに
-   Enhance: エンドポイント`i/webhook/update`の必須項目を`webhookId`のみに
-   Enhance: エンドポイント`admin/ad/update`の必須項目を`id`のみに
- Enhance: `default.yml`内の`url`, `db.db`, `db.user`,
`db.pass`を環境変数から読み込めるように
- Enhance:
エンドポイント`api/meta`にプロパティ`noteSearchableScope`が増え、`string`値`local`または`global`を返却します
- Fix:
チャート生成時にinstance.suspensionStateに置き換えられたinstance.isSuspendedが参照されてしまう問題を修正
- Fix: ユーザーのフィードページのMFMをHTMLに展開するように
([#&#8203;14006](https://github.com/misskey-dev/misskey/issues/14006))
- Fix: アンテナ・クリップ・リスト・ウェブフックがロールポリシーの上限より一つ多く作れてしまうのを修正
([#&#8203;14036](https://github.com/misskey-dev/misskey/issues/14036))
- Fix: notRespondingSinceが実装される前に不通になったインスタンスが自動的に配信停止にならない
([#&#8203;14059](https://github.com/misskey-dev/misskey/issues/14059))
- Fix:
FTT有効時、タイムライン用エンドポイントで`sinceId`にキャッシュ内最古のものより古いものを指定した場合に正しく結果が返ってこない問題を修正
-   Fix: 自分以外のクリップ内のノート個数が見えることがあるのを修正
-   Fix: 空文字列のリアクションはフォールバックされるように
-   Fix: リノートにリアクションできないように
-   Fix: ユーザー名の前後に空白文字列がある場合は省略するように
-   Fix: プロフィール編集時に名前を空白文字列のみにできる問題を修正
- Fix: ユーザ名のサジェスト時に表示される内容と順番を調整(以下の順番になります)
[#&#8203;14149](https://github.com/misskey-dev/misskey/issues/14149)

    1.  フォロー中かつアクティブなユーザ
    2.  フォロー中かつ非アクティブなユーザ
    3.  フォローしていないアクティブなユーザ
    4.  フォローしていない非アクティブなユーザ

    また、自分自身のアカウントもサジェストされるようになりました。
-   Fix: 一般ユーザーから見たユーザーのバッジの一覧に公開されていないものが含まれることがある問題を修正\
(Cherry-picked from
[https://github.com/MisskeyIO/misskey/pull/652](https://github.com/MisskeyIO/misskey/pull/652))
-   Fix: ユーザーのリアクション一覧でミュート/ブロックが機能していなかった問題を修正
-   Fix: FTT有効時にリモートユーザーのノートがHTLにキャッシュされる問題を修正
-   Fix: 一部の通知がローカル上のリモートユーザーに対して行われていた問題を修正
- Fix: エラーメッセージの誤字を修正
([#&#8203;14213](https://github.com/misskey-dev/misskey/issues/14213))
-   Fix: ソーシャルタイムラインにローカルタイムラインに表示される自分へのリプライが表示されない問題を修正
-   Fix: リノートのミュートが適用されるまでに時間がかかることがある問題を修正\
(Cherry-picked from
Type4ny-Project/Type4ny@e960102)
- Fix: Steaming APIが不正なデータを受けた場合の動作が不安定である問題
[#&#8203;14251](https://github.com/misskey-dev/misskey/issues/14251)
-   Fix: `users/search`において `@` から始まる文字列が与えられた際の処理が正しくなかった問題を修正
    -   名前や自己紹介に `@` から始まる文言が含まれるユーザーも検索できるようになります
-   Fix: 一部のMisskey以外のソフトウェアからファイルを受け取れない問題
(Cherry-picked
f[https://github.com/Secineralyr/misskey.dream/pull/73](https://github.com/Secineralyr/misskey.dream/pull/73)l/73/commits/652eaff1e8aa00b890d71d2e1e52c263c1e67c76)
    -   NOTE: `drive_file`の`url`, `uri`, `src`の上限が512から1024に変更されます
        Migrationではカラム定義の変更のみが行われます。
サーバー管理者は各サーバーの必要に応じ`drive_file`
`("uri")`に対するインデックスを張りなおすことでより安定しDBの探索が行われる可能性があります。詳細 は
[GitHub](https://github.com/misskey-dev/misskey/pull/14323#issuecomment-2257562228)で確認可能です
-   Fix: 自分のフォロワー限定投稿に対するリプライがホームタイムラインで見えないことが有る問題を修正
-   Fix: フォローしていないユーザによるフォロワー限定投稿に対するリプライがソーシャルタイムラインで表示されることがある問題を修正

##### Misskey.js

-   Feat: `/drive/files/create` のリクエストに対応(`multipart/form-data`に対応)
-   Feat: `/admin/role/create` のロールポリシーの型を修正

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://github.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC4xNS4wIiwidXBkYXRlZEluVmVyIjoiMzguMTUuMCIsInRhcmdldEJyYW5jaCI6Im1hc3RlciIsImxhYmVscyI6WyJhdXRvbWVyZ2UiLCJ1cGRhdGUvZG9ja2VyL2dlbmVyYWwvbm9uLW1ham9yIl19-->
taiyme added a commit that referenced this pull request Aug 12, 2024
* fix(frontend): 直前のパターンを記録するように

* fix(frontend): フォーカス/タブ移動に関する挙動を調整 (#226)

Cherry-pick commit e8c0306

Co-authored-by: taiyme <53635909+taiyme@users.noreply.github.com>

* focusのデザイン修正

* move scripts

* Modalにfocus trapを追加

* 記録するホットキーはレートリミット式にする

* escキーのハンドリングをMkModalに統一

* fix

* enterで子メニューを開けるように

* lint

* fix focus trap

* improve switch accessibility

* 一部のmodalのフォーカストラップが外れない問題を修正

* fix

* fix

* Revert "記録するホットキーはレートリミット式にする"

This reverts commit 40a7509.

* Revert "fix(frontend): 直前のパターンを記録するように"

This reverts commit 5372b25.

* Revert "Revert "fix(frontend): 直前のパターンを記録するように""

This reverts commit a9bb52e.

* Revert "Revert "記録するホットキーはレートリミット式にする""

This reverts commit bdac342.

* 試験的にCypressでのFocustrapを無効化

* fix

* fix focus-trap

* Update Changelog

* ✌️

* fix focustrap invocation logic

* スクロールがsticky headerを考慮するように

* 🎨

* スタイルの微調整

* 🎨

* remove deprecated key aliases

* focusElementが足りなかったので修正

* preview系にfocus時スタイルが足りなかったので修正

* `returnFocusElement` -> `returnFocusTo`

* lint

* Update packages/frontend/src/components/MkModalWindow.vue

* Apply suggestions from code review

Co-authored-by: taiy <53635909+taiyme@users.noreply.github.com>

* keydownイベントをまとめる

* use correct pesudo-element selector

* fix

* rename

---------

Co-authored-by: taiyme <53635909+taiyme@users.noreply.github.com>
Co-authored-by: syuilo <4439005+syuilo@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants