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

Implement highlighting of active feed item and more keyboard shortcuts #2677

Merged
merged 5 commits into from
Jun 8, 2024

Conversation

pbek
Copy link
Member

@pbek pbek commented Jun 2, 2024

To help with #2671.

Summary

  • Implement j and k keyboards shortcuts for navigating through feed items
  • Implement s, i and l keyboards shortcuts for staring current feed item
  • Implement o keyboards shortcut for opening the URL of current feed item
  • Implement u keyboards shortcut for marking current feed item read/unread
  • Implement highlighting of active feed item

Signed-off-by: Patrizio Bekerle <patrizio@bekerle.com>
@pbek pbek requested review from SMillerDev and Grotax June 2, 2024 19:05
@pbek pbek marked this pull request as ready for review June 2, 2024 19:06
@pbek
Copy link
Member Author

pbek commented Jun 2, 2024

Needed a little trickery...

@pbek pbek changed the title Implement highlighting of active feed item Implement highlighting of active feed item and more keyboard shortcuts Jun 3, 2024
@pbek
Copy link
Member Author

pbek commented Jun 3, 2024

Meanwhile, I added a few more keyboard shortcuts

Signed-off-by: Patrizio Bekerle <patrizio@bekerle.com>
@Grotax
Copy link
Member

Grotax commented Jun 3, 2024

Cool, code looks ok to me. I will try it when I have time to make sure :)

@pbek
Copy link
Member Author

pbek commented Jun 4, 2024

Cool, code looks ok to me. I will try it when I have time to make sure :)

At least the "trying" is one git checkout and make term away now. 😆

@pbek
Copy link
Member Author

pbek commented Jun 7, 2024

I spent so many hours in bringing the current item into view in the VirtualScroll. It's not meant to allow to scroll to an element. ☹️

@Grotax
Copy link
Member

Grotax commented Jun 8, 2024

Just tested this, works nice.

But yea it seems like VirtualScroll is not the solution for news. It has this other issue being very jumpy when scrolling.

If you feel done with this (at the moment) we can merge this

@Grotax
Copy link
Member

Grotax commented Jun 8, 2024

pbek added 2 commits June 8, 2024 12:29
Signed-off-by: Patrizio Bekerle <patrizio@bekerle.com>
Signed-off-by: Patrizio Bekerle <patrizio@bekerle.com>
@pbek
Copy link
Member Author

pbek commented Jun 8, 2024

Ah two commits miss the sign-off https://github.com/nextcloud/news/pull/2677/checks?check_run_id=25958808201

Sorry, I fixed that now.

Just tested this, works nice.

Thank you!

But yea it seems like VirtualScroll is not the solution for news. It has this other issue being very jumpy when scrolling.

Have you seen anything in another app that is more to your liking?

If you feel done with this (at the moment) we can merge this

Yes, I'm done with that PR. Thank you!

@Grotax Grotax merged commit 8145582 into nextcloud:master Jun 8, 2024
12 of 21 checks passed
@Grotax
Copy link
Member

Grotax commented Jun 8, 2024

Have you seen anything in another app that is more to your liking?

No, I'm not a big user of the nextcloud or even news in the web. I mostly use the android apps.

related: #2498

@pbek
Copy link
Member Author

pbek commented Jun 8, 2024

No, I'm not a big user of the nextcloud or even news in the web. I mostly use the android apps.

Interesting... Which Android App do you use? I use (Nextcloud) News on Android. But I read mostly on the web directly.

related: #2498

Hm, then maybe just a container with infinite scroll?

@Grotax
Copy link
Member

Grotax commented Jun 8, 2024

Interesting... Which Android App do you use? I use (Nextcloud) News on Android. But I read mostly on the web directly.

Same https://github.com/nextcloud/news-android

Grotax added a commit that referenced this pull request Jun 10, 2024
Changed
- added alternative development environment (#2670)
- Implement `j` and `k` keyboards shortcuts for navigating through feed items (#2671)
- Implement `s`, `i` and `l` keyboards shortcuts for staring current feed item (#2677)
- Implement `o` keyboards shortcut for opening the URL of current feed item (#2677)
- Implement `u` keyboards shortcut for marking current feed item read/unread (#2677)
- Implement highlighting of active feed item (#2677)

Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
@Grotax Grotax mentioned this pull request Jun 10, 2024
Grotax added a commit that referenced this pull request Jun 11, 2024
Changed
- added alternative development environment (#2670)
- Implement `j` and `k` keyboards shortcuts for navigating through feed items (#2671)
- Implement `s`, `i` and `l` keyboards shortcuts for staring current feed item (#2677)
- Implement `o` keyboards shortcut for opening the URL of current feed item (#2677)
- Implement `u` keyboards shortcut for marking current feed item read/unread (#2677)
- Implement highlighting of active feed item (#2677)

Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
@Grotax Grotax mentioned this pull request Jun 23, 2024
19 tasks
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.

2 participants