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): 画面ビューワをタップした場合、マウスクリックと同様に画像ビューワを閉じるように #11211

Merged
merged 3 commits into from
Jul 10, 2023

Conversation

akanevrc
Copy link
Contributor

@akanevrc akanevrc commented Jul 9, 2023

What

スマホ版のWebページでノート内の画像表示をクローズする操作が変わります。
以前は、クローズボタンまたはスワイプのみでクローズしていました。
以後は、画像または画像外の領域をタップすることでもクローズできるようになります。

Why

画像表示モードのクローズに関して、PC版と大きくUXが異なっていました。
このPRによりPC版とスマホ版が同様のUXになります。

Additional info (optional)

Issue

#11210

テスト観点

実際に操作を行なって、UIの動作に問題がないか確認する必要があります。

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

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

codecov bot commented Jul 9, 2023

Codecov Report

Merging #11211 (0ae1a05) into develop (5059d4d) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop   #11211      +/-   ##
===========================================
- Coverage    77.70%   77.70%   -0.01%     
===========================================
  Files          908      908              
  Lines        91750    91758       +8     
  Branches      7670     7672       +2     
===========================================
+ Hits         71295    71296       +1     
- Misses       20455    20462       +7     
Impacted Files Coverage Δ
packages/frontend/src/components/MkMediaList.vue 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

@tamaina tamaina changed the title Fix(frontend): スマホで画像表示時に画面をタップして画像表示をクローズすることができない仕様を修正 fix(frontend): 画面ビューワをタップした場合、マウスクリックと同様に画像ビューワを閉じるように Jul 10, 2023
@tamaina tamaina merged commit 63e21a4 into misskey-dev:develop Jul 10, 2023
@tamaina
Copy link
Contributor

tamaina commented Jul 10, 2023

👍 👍👍👍

anatawa12 pushed a commit to anatawa12/misskey that referenced this pull request Jul 13, 2023
…1211)

* fix: change tapAction of photoswipe to 'close'

* doc: update CHANGELOG.md

---------

Co-authored-by: tamaina <[email protected]>
Cherry-pics: 63e21a4
slofp pushed a commit to Secineralyr/misskey.dream that referenced this pull request Jul 21, 2023
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.

2 participants