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

refactor(neon_news, neon_notes, neon_notifications): replace long press actions with swipe-to-dismiss #2241

Merged
merged 1 commit into from
Jul 16, 2024

Conversation

satvik2131
Copy link
Contributor

@satvik2131 satvik2131 commented Jul 4, 2024

#2158

Implemented swipe-to-dismiss feature in :

  1. Neon News

    • Articles (swipe to mark read/unread)
    • Folders
    • Feeds
  2. Neon Notifications

  3. Neon notes

@provokateurin
Copy link
Member

Thanks for the work btw :)

@satvik2131
Copy link
Contributor Author

Thanks for the work btw :)

most welcome , will be updating the changes soon, and btw you are literally very active , thanks for the feedback and guidance :)

Copy link
Member

@provokateurin provokateurin left a comment

Choose a reason for hiding this comment

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

Thank you so much!
You need to fix the DCO and please name the commit something like refactor(neon_news,neon_notes,neon_notifications): Replace long press actions with swipe-to-dismiss

@satvik2131 satvik2131 changed the title feat: Added swipe-to-dismiss #2158 refactor(neon_news, neon_notes, neon_notifications): replace long press actions with swipe-to-dismiss #2158 Jul 4, 2024
@satvik2131 satvik2131 force-pushed the main branch 2 times, most recently from d0ad239 to e2422d2 Compare July 4, 2024 17:51
@provokateurin
Copy link
Member

You only changed the PR title, not the commit message. Also please don't mention the issue, you can just put it in the PR description to refer to the issue.

@satvik2131 satvik2131 changed the title refactor(neon_news, neon_notes, neon_notifications): replace long press actions with swipe-to-dismiss #2158 refactor(neon_news, neon_notes, neon_notifications): replace long press actions with swipe-to-dismiss Jul 4, 2024
@satvik2131 satvik2131 force-pushed the main branch 2 times, most recently from 665a48c to 0b1abf9 Compare July 4, 2024 18:33
@provokateurin provokateurin linked an issue Jul 4, 2024 that may be closed by this pull request
@provokateurin provokateurin enabled auto-merge July 4, 2024 18:45
@provokateurin
Copy link
Member

Please checkout the issues the CI found. You can run melos run format to fix the formatting problems, but the analyzer issues need to be fixed as well.

auto-merge was automatically disabled July 5, 2024 03:05

Head branch was pushed to by a user without write access

@satvik2131 satvik2131 force-pushed the main branch 2 times, most recently from d8e20c1 to 5b32b71 Compare July 5, 2024 12:25
… actions with swipe-to-dismiss

Signed-off-by: satvik2131 <satvik213161@gmail.com>
Signed-off-by: provokateurin <kate@provokateurin.de>
@provokateurin provokateurin enabled auto-merge July 16, 2024 14:46
@provokateurin provokateurin merged commit b238891 into nextcloud:main Jul 16, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use swipe-to-dismiss instead of long press on ListTiles
2 participants