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

enhance(client): photoswipe 表示時に戻る操作をしても前の画面に戻らないように #10098

Merged
merged 3 commits into from
Feb 26, 2023

Conversation

Khsmty
Copy link
Contributor

@Khsmty Khsmty commented Feb 26, 2023

Resolve #10086

photoswipe を表示している際に、戻る操作をすると photoswipe を閉じるようにし、誤って前の画面に戻ることがないようにしました。

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Feb 26, 2023
@codecov
Copy link

codecov bot commented Feb 26, 2023

Codecov Report

Merging #10098 (3f619dd) into develop (504baa9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop   #10098   +/-   ##
========================================
  Coverage    25.06%   25.06%           
========================================
  Files          705      705           
  Lines        65150    65150           
  Branches      2307     2307           
========================================
  Hits         16328    16328           
  Misses       48822    48822           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@syuilo
Copy link
Member

syuilo commented Feb 26, 2023

Aページ->Bページ→Aページ(ブラウザバック)と遷移してきた状態で一度でも画像を開いてしまうとBに(→クリックで)戻れなくなるのは微妙かも?

@Khsmty
Copy link
Contributor Author

Khsmty commented Feb 26, 2023

確かにそうですね 🤔
履歴に残さない方法を考えてみます

@Khsmty
Copy link
Contributor Author

Khsmty commented Feb 26, 2023

他の画像ビューワーとかの実装とか見てみても同じような実装なので無理なんですかねぇ…

@momoirodouhu
Copy link
Contributor

momoirodouhu commented Feb 26, 2023

戻るイベントで#pspwになる
popstat受信で画像が閉じる
#pspwなのでさらに履歴を戻される

って見た感じなりそうだけど大丈夫なんだろうか

Bに戻れなくなる

pushstatは履歴を追加してるだけだからうしろに元の履歴が残ってるはず

@Khsmty
Copy link
Contributor Author

Khsmty commented Feb 26, 2023

pushstatは履歴を追加してるだけだからうしろに元の履歴が残ってるはず

pushstate で後ろに履歴追加するので元の履歴は残らないですね

@syuilo
Copy link
Member

syuilo commented Feb 26, 2023

Aページ->Bページ→Aページ(ブラウザバック)と遷移してきた状態で一度でも画像を開いてしまうとBに(→クリックで)戻れなくなるのは微妙かも?

頻繁に発生するシチュエーションではないと思うわれるのでマージします

@syuilo syuilo merged commit 72888b4 into misskey-dev:develop Feb 26, 2023
@syuilo
Copy link
Member

syuilo commented Feb 26, 2023

👍

@momoirodouhu
Copy link
Contributor

テキトウなこと言ってしまってすいません

syuilo added a commit that referenced this pull request Feb 26, 2023
* Add dialog to remove follower (#9718)

* update PULL_REQUEST_TEMPLATE

* 起動時にRedisの疎通確認を行う (#9832)

* 起動時にRedisの疎通確認を行う

* check:connectをstart内に移動

---------

Co-authored-by: tamaina <[email protected]>

* Pass `--detectOpenHandles` to Jest (#9895)

Co-authored-by: tamaina <[email protected]>

* enhance(client): MkUrlPreviewの閉じるボタンを見やすく (#9913)

Co-authored-by: tamaina <[email protected]>

* test(backend): restore ap-request tests (#9997)

Co-authored-by: tamaina <[email protected]>

* fix/refaftor(client): MkTime.vueの変更 (#10061)

* fix(client): MkTime.timeにstringでもDateでない値が入った場合、?を表示

* fix(client): MkTimeを改良

* numberを許容

* falsyな値もとる

* 不明

* ありません

* fix

* fix(server): notes/createで、fileIdsと見つかったファイルの数が異なる場合はエラーにする (#9911)

* fix(server): notes/createで、fileIdsと見つかったファイルの数が異なる場合はエラーにする

* NO_SUCH_FILE

* Update codecov.yml

* Update apple-touch-icon.png

* デプロイされているプレビュー環境がない場合はプレビュー環境を削除しないようにする (#10062)

* デプロイされているプレビュー環境がない場合はDestroy preview environmentを実行しないようにする

* CIがない場合の処理追加

* enhance(client): improve clip menu ux

* 未知のユーザーが deleteActor されたら処理をスキップする (#10067)

* fix(client): Android ChromeでPWAとしてインストールできない問題を修正 (#10069)

* fix(client): Android ChromeでPWAとしてインストールできない問題を修正

* 順番関係ある?

* Windows環境でswcを使うと正常にビルドができない問題の修正 (#10074)

* Update @swc/core to v1.3.36

* Update CHANGELOG.md

* Update CHANGELOG.md

* バックグラウンドで一定時間経過したらページネーションのアイテム更新をしない (#10053)

* 🎨

* feat: 2つの検索画面の統合 (#9949) (#10038)

* feat: 検索画面の UI を統一

* fix: エラーの修正

* add: changelog

---------

Co-authored-by: syuilo <[email protected]>

* enhance(client): ノートメニューからユーザーメニューを開けるように

Resolve #10019

* enhance(client): renoteした際の表示を改善

Resolve #10078

* Update CHANGELOG.md

* enhance(client): tweak contextmenu position calculation

* 🎨

* 🎨

* feat: in-channel featured note

Resolve #9938

* refactor(frontend): fix eslint error (#10084)

* Simplify search.vue (remove dead code) (#10088)

* Simplify search.vue

This is already handled by the code above it, no need to handle it twice

* Remove unused imports

* Update about-misskey.vue

* test(server): add validation test of api:notes/create (#10090)

* fix(server): notes/createのバリデーションが効いていない
Fix #10079

Co-Authored-By: mei23 <[email protected]>

* anyOf内にバリデーションを書いても最初の一つしかチェックされない

* ✌️

* wip

* wip

* ✌️

* RequiredProp

* Revert "RequiredProp"

This reverts commit 7469390.

* add api:notes/create

* fix lint

* text

* ✌️

* improve readability

---------

Co-authored-by: mei23 <[email protected]>
Co-authored-by: syuilo <[email protected]>

* New Crowdin updates (#10059)

* New translations ja-JP.yml (Japanese, Kansai)

* New translations ja-JP.yml (Romanian)

* New translations ja-JP.yml (French)

* New translations ja-JP.yml (Spanish)

* New translations ja-JP.yml (Arabic)

* New translations ja-JP.yml (Czech)

* New translations ja-JP.yml (German)

* New translations ja-JP.yml (Italian)

* New translations ja-JP.yml (Korean)

* New translations ja-JP.yml (Polish)

* New translations ja-JP.yml (Russian)

* New translations ja-JP.yml (Slovak)

* New translations ja-JP.yml (Ukrainian)

* New translations ja-JP.yml (Chinese Simplified)

* New translations ja-JP.yml (Chinese Traditional)

* New translations ja-JP.yml (English)

* New translations ja-JP.yml (Vietnamese)

* New translations ja-JP.yml (Indonesian)

* New translations ja-JP.yml (Bengali)

* New translations ja-JP.yml (Thai)

* New translations ja-JP.yml (Japanese, Kansai)

* New translations ja-JP.yml (Chinese Traditional)

* New translations ja-JP.yml (Chinese Traditional)

* New translations ja-JP.yml (German)

* New translations ja-JP.yml (English)

* New translations ja-JP.yml (German)

* New translations ja-JP.yml (Ukrainian)

* New translations ja-JP.yml (Chinese Simplified)

* New translations ja-JP.yml (English)

* New translations ja-JP.yml (Ukrainian)

* New translations ja-JP.yml (Ukrainian)

* New translations ja-JP.yml (Ukrainian)

* New translations ja-JP.yml (Chinese Simplified)

* New translations ja-JP.yml (Japanese, Kansai)

* New translations ja-JP.yml (Thai)

* New translations ja-JP.yml (Thai)

* New translations ja-JP.yml (Thai)

* New translations ja-JP.yml (Spanish)

* New translations ja-JP.yml (Spanish)

* enhance(client): improve user menu ux

* enhance(client): photoswipe 表示時に戻る操作をしても前の画面に戻らないように (#10098)

* enhance(client): photoswipe 表示時に戻る操作をしても前の画面に戻らないように

* add: changelog

---------

Co-authored-by: syuilo <[email protected]>

* enhance(client): メニューの「もっと」からインスタンス情報を見れるように

* [Fix] fixed an typo in error message (#10102)

* Update codecov.yml

* Update CHANGELOG.md

* fix(server): エラーのスタックトレースは返さないように

Fix #10064

* [chore]Editorconfig: ymlに加えてyamlファイルに対しても同じ規約を適用する (#10081)

* Added yaml file in addition to yml file, in editorconfig

* Applied editorconfig for pnpm-workspace.yaml

---------

Co-authored-by: syuilo <[email protected]>

* update deps

* ホームタイムラインの読み込みでクエリタイムアウトになるのを修正する (#10106)

* refactor

* New translations ja-JP.yml (French) (#10103)

* Update CHANGELOG.md

* 13.8.0

---------

Co-authored-by: atsuchan <[email protected]>
Co-authored-by: Masaya Suzuki <[email protected]>
Co-authored-by: tamaina <[email protected]>
Co-authored-by: Kagami Sascha Rosylight <[email protected]>
Co-authored-by: taiy <[email protected]>
Co-authored-by: xianon <[email protected]>
Co-authored-by: kabo2468 <[email protected]>
Co-authored-by: YS <[email protected]>
Co-authored-by: Khsmty <[email protected]>
Co-authored-by: Soni L <[email protected]>
Co-authored-by: mei23 <[email protected]>
Co-authored-by: daima3629 <[email protected]>
Co-authored-by: Windymelt <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/frontend Client side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

画像を表示している時に、戻るボタンで前に戻らないようにしてほしい
3 participants