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

Limit touch click detection to the content div #1402

Merged
merged 2 commits into from
Jan 31, 2025

Conversation

BentiGorlich
Copy link
Member

  • only open a post when clicking the content section (solves opening the post when trying to click the "more" dropdown or clicking reply)
  • preventDefault() when clicking the expand content button

- only open a post when clicking the content section (solves opening the post when trying to click the "more" dropdown or clicking reply)
- `preventDefault()` when clicking the expand content button
@BentiGorlich BentiGorlich added frontend Visual issues, improvements, bugs or other aspects relating mostly to the front end backend Backend related issues and pull requests labels Jan 31, 2025
@BentiGorlich BentiGorlich added this to the v1.8.0 milestone Jan 31, 2025
@BentiGorlich BentiGorlich self-assigned this Jan 31, 2025
@BentiGorlich BentiGorlich requested a review from melroy89 January 31, 2025 09:36
@melroy89
Copy link
Member

There is a twig audit package failure.

@BentiGorlich BentiGorlich enabled auto-merge (squash) January 31, 2025 12:21
@BentiGorlich BentiGorlich merged commit aec3229 into main Jan 31, 2025
7 checks passed
@BentiGorlich BentiGorlich deleted the fix/touch-click-to-aggressive branch January 31, 2025 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Backend related issues and pull requests frontend Visual issues, improvements, bugs or other aspects relating mostly to the front end
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants