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: iPadでdeck uiでマウスホイールでスクロールできない #15244

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

anatawa12
Copy link
Member

What

iPadOSでマウスホイールを使用してスクロールができない問題を修正しました。

また、水平スクロールができない問題の修正のため、マウスホイールでスクロールしてるときには snap を無効化するようにしました。

Fix #11733
Relates #7220?

Why

Additional info (optional)

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 Jan 9, 2025
@anatawa12 anatawa12 changed the title fix: ipadでdeck uiでスクロールできない fix: iPadでdeck uiでマウスホイールでスクロールできない Jan 9, 2025
Copy link

codecov bot commented Jan 9, 2025

Codecov Report

Attention: Patch coverage is 0% with 15 lines in your changes missing coverage. Please review.

Project coverage is 18.82%. Comparing base (9c70a4e) to head (5ea0f04).

Files with missing lines Patch % Lines
packages/frontend/src/ui/deck.vue 0.00% 14 Missing ⚠️
packages/frontend/src/ui/deck/column.vue 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #15244       +/-   ##
============================================
- Coverage    39.40%   18.82%   -20.59%     
============================================
  Files         1607      764      -843     
  Lines       210127   111056    -99071     
  Branches      3785     1233     -2552     
============================================
- Hits         82803    20907    -61896     
+ Misses      126718    89562    -37156     
+ Partials       606      587       -19     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

anatawa12 added a commit to anatawa12/misskey that referenced this pull request Jan 28, 2025
commit b299046
Author: anatawa12 <[email protected]>
Date:   Thu Jan 9 20:23:25 2025 +0900

    docs(changelog): iPadOSでdeck uiをマウスカーソルによってスクロールできない問題を修正

commit 8a531b7
Author: anatawa12 <[email protected]>
Date:   Thu Jan 9 20:11:34 2025 +0900

    fix: ipadでdeck uiでスクロールできない
@kakkokari-gtyih kakkokari-gtyih added this to the v2025.2.1 milestone Feb 1, 2025
}

document.addEventListener('pointerdown', pointerEvent);
document.addEventListener('pointermove', pointerEvent);
Copy link
Member

Choose a reason for hiding this comment

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

パフォーマンスへの影響が気になるわね

Copy link
Member Author

Choose a reason for hiding this comment

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

pointerDown / upだけでいい説はありますね。ちょっと試してみます。

Copy link
Member Author

Choose a reason for hiding this comment

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

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
Development

Successfully merging this pull request may close these issues.

iPad Safariかつデッキで、トラックパッドのスクロールが効かない
3 participants