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

Missing features in News 25.x #2503

Closed
17 of 19 tasks
Grotax opened this issue Dec 17, 2023 · 49 comments
Closed
17 of 19 tasks

Missing features in News 25.x #2503

Grotax opened this issue Dec 17, 2023 · 49 comments
Labels
frontend impact Javascript/Frontend code
Milestone

Comments

@Grotax
Copy link
Member

Grotax commented Dec 17, 2023

Background

News 25.x includes a major rewrite of the fronted. This change was necessary since the old UI was based on angular JS (the original) and that framework is no longer supported by Google and therefore also all dependencies and everything around it.

The new frontend uses vue js and the nextcloud libs for vue js.

See https://github.com/orgs/nextcloud/projects/85 for the things that have been done so far.

Current Situation

We have alpha releases that include basic functionality. People can install them manually from the app-store via cli or however they like.

Missing Features

Pull requests welcome ❤️

@Grotax Grotax added the frontend impact Javascript/Frontend code label Dec 17, 2023
@Grotax Grotax pinned this issue Dec 17, 2023
@Grotax Grotax mentioned this issue Dec 17, 2023
3 tasks
@SMillerDev SMillerDev added this to the Vue Rewrite milestone Dec 17, 2023
@alxrdn
Copy link

alxrdn commented Jan 17, 2024

Hello,
following this discussion #2423 (comment)
Can the compact view mode be listed here as a missing feature?
I unfortunately don't have the competencies to provide with a PR for this, but someone probably do have

@kbridger
Copy link

Are there features missing that will impact external apps accessing news, eg. Nextcloud News Android app? Or is this all related to the News UI in Nextcloud via web browser?

@Grotax
Copy link
Member Author

Grotax commented Feb 12, 2024

All related to the frontend, the API/backend which is used by the apps actually is a bit better, performance, bug fixes for edge cases and so on

@PhieF

This comment has been minimized.

@Grotax

This comment has been minimized.

@pierfu

This comment was marked as duplicate.

@zachpmanson
Copy link

Very happy with the rewrite, much more responsive than the AngularJS version.

Missing feature: there is no way to move a feed from one folder to another

@azmeuk

This comment has been minimized.

@SMillerDev

This comment has been minimized.

@zachpmanson

This comment has been minimized.

@SMillerDev

This comment has been minimized.

@FadeFx
Copy link

FadeFx commented May 18, 2024

Hi, i have to add that scrolling in the mobile Interface ist a Pita, as items disapear from the list too early and the rest of the list jumps up...

Also I'd love to see the full post list again, I was used to scroll through the list of full news items, as most providers do not feed the full article anyway.

@Grotax
Copy link
Member Author

Grotax commented May 18, 2024

For the first issue you mention there is already #2498 basically the vue component in use does not work properly.

I'm not sure if that view will come back. There are so many different opinions how the views should be done.

@SMillerDev
Copy link
Contributor

Unless people make a pull request of course, then there's a reasonable chance it will be added

@ChristianKrausse
Copy link

I miss the Keybord Navigation to the next Article

@pbek
Copy link
Member

pbek commented Jun 4, 2024

I miss the Keybord Navigation to the next Article

See #2671.

@pbek

This comment was marked as off-topic.

@Grotax

This comment was marked as off-topic.

@pbek

This comment was marked as off-topic.

@ax1036

This comment was marked as off-topic.

@SMillerDev

This comment was marked as off-topic.

@pbek

This comment was marked as off-topic.

@AHeinlein
Copy link

It might be worth it to have an "officially recommended" solution to this for now, apart from uninstalling news.
There is a way to install 25alpha and there is a workaround to get 24.x running und Nextcloud 28 (#2610). I am using the workaround for almost half a year now and it works well for me. I wish I could help more, but my time is limited and my php skills are, well, basic at most.

@Grotax
Copy link
Member Author

Grotax commented Jun 16, 2024

My official recommended solution is to update to the alpha. We do not support older versions at all.

Some people might find this strange but that's how it is in open source sometimes.

@Grotax
Copy link
Member Author

Grotax commented Sep 17, 2024

Should be up to date but if you want to work on the code and implement create a PR better double check 😊

@pbek
Copy link
Member

pbek commented Oct 15, 2024

@Grotax, is Reverse Order (Oldest to Newest) the same as Sorting by date does not work (#2678)?

@pbek
Copy link
Member

pbek commented Oct 15, 2024

And do you think we could add Option in folders to hide feeds with no unread items to the list?

Grotax added a commit that referenced this issue Oct 19, 2024
Fixed
- First features for user settings after vue migration (#2795)
- scroll stops after first bunch of items when they are marked read during scroll (#2814)
- marked read items disappear when showAll is disabled in folder or feed view (#2814)
- Feed item lists are merged when filtered (#2516)
- Mark as read on Scroll (#2503)
- Open news with the last viewed feed or folder (#2507)

Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
@Grotax
Copy link
Member Author

Grotax commented Oct 19, 2024

@Grotax, is Reverse Order (Oldest to Newest) the same as Sorting by date does not work (#2678)?

Yea that is the same feature basically both want to have sorting by date and that the user can decide if old or new first

@wofferl
Copy link
Collaborator

wofferl commented Oct 19, 2024

@Grotax

Reverse Order (Oldest to Newest), Sorting by date does not work Sorting by date does not work #2678* [ ]

this is implemented with #2795

Option in folders to hide feeds with no unread items (Maybe add hint somewhere in the UI so that users notice it is active and do not create "I can't see anything" issues.)

Should this really be an own option or like in news24 when showAll is disabled autohide read folders

Grotax added a commit that referenced this issue Oct 19, 2024
Fixed
- First features for user settings after vue migration (#2795)
- scroll stops after first bunch of items when they are marked read during scroll (#2814)
- marked read items disappear when showAll is disabled in folder or feed view (#2814)
- Feed item lists are merged when filtered (#2516)
- Mark as read on Scroll (#2503)
- Open news with the last viewed feed or folder (#2507)

Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
@Grotax
Copy link
Member Author

Grotax commented Oct 19, 2024

I think how it was in news24 is ok

@pbek
Copy link
Member

pbek commented Oct 19, 2024

Reverse Order (Oldest to Newest), Sorting by date does not work #2678* [ ]

this is implemented with #2795

Are you sure this already worked? When I tested yesterday, I didn't manage to get the correct order in feeds. 🤔
...especially with infinite scrolling.

I worked on sorting in #2811.

@wofferl
Copy link
Collaborator

wofferl commented Oct 19, 2024

Reverse Order (Oldest to Newest), Sorting by date does not work #2678* [ ]

this is implemented with #2795

Are you sure this already worked? When I tested yesterday, I didn't manage to get the correct order in feeds. 🤔 ...especially with infinite scrolling.

I worked on sorting in #2811.

Yes this worked with alpha11. When you set the global setting to "Reverse ordering" the feeds are sorted from oldest to newest.

@pbek
Copy link
Member

pbek commented Oct 19, 2024

Ok, global settings in something different then. I was working on the per-feed settings.

@wofferl
Copy link
Collaborator

wofferl commented Oct 19, 2024

Ok, global settings in something different then. I was working on the per-feed settings.

Yes, we talked about this in your PR :)

@FadeFx

This comment has been minimized.

@cloudymax
Copy link

is importing OPML files an abandoned feature? No longer works in the app and missing from web ui

@SMillerDev
Copy link
Contributor

is importing OPML files an abandoned feature? No longer works in the app and missing from web ui

It's in the list above, so clearly not.

@SMillerDev
Copy link
Contributor

I have two requests,
1.) When in a news item, klicking back or using the back gesture on a mobile device, should actually close the item instead of loading the previous page.

2.) As Nextcloud has for a while now disabled the pull down to reload gesture completely, it would be cool if you, like suggested by nextcloud dev, could enable that for News to reload the page and check for new items.
I would BTW prefere this instead of reloading in background, as the latter would cause, depending on sorting order, content to jump away unexpectedly.
#2502

These seem both like new feature requests, please file those in Discussions or as a pull request.

Grotax added a commit that referenced this issue Oct 23, 2024
Changed
- Feed ordering will now be used for sorting items (#2678)

Fixed
- restore old scroll behavior to scroll after articles needed for mark scroll on read (#2817)
- 25.0.0-alpha2: Feed folder bar menu always opens collapsed (#2508)
- fix: hide folder/feeds with no unread items when `showAll` is disabled (#2503)
- Items keep popping up in Folder, All or Unread view (#2833)

Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
Grotax added a commit that referenced this issue Oct 23, 2024
Changed
- Feed ordering will now be used for sorting items (#2678)

Fixed
- restore old scroll behavior to scroll after articles needed for mark scroll on read (#2817)
- 25.0.0-alpha2: Feed folder bar menu always opens collapsed (#2508)
- fix: hide folder/feeds with no unread items when `showAll` is disabled (#2503)
- Items keep popping up in Folder, All or Unread view (#2833)

Signed-off-by: Benjamin Brahmer <info@b-brahmer.de>
@Ekleog
Copy link

Ekleog commented Nov 1, 2024

Just for my edification, are all the features listed here blockers for the 25.x release? In particular, I'm under the impression that things like the compact mode could reasonably be postponed to 26.x, so that nextcloud news can be used again with fewer features in the meantime?

For some context, Nextcloud 27 has been EOL for a few months, so there's currently no supported way of installing of Nextcloud News, despite it being a featured app on the official website — an alpha install is not really a supported install IMHO.

Regardless of your choice on the matter, thank you for all the work you put towards Nextcloud News 25! I'm very hyped about it, even though I've been living without an RSS feed for quite a few months now 😅

@MightyCreak
Copy link

To be fair, if you have access to the console, it's fairly easy to install the alpha version. But I agree it's a workaround and I'd prefer an official release of 25.0 with less features, knowing more are coming in 25.x afterwards

@Grotax
Copy link
Member Author

Grotax commented Nov 19, 2024

Closing this as we are about to release 25.0.0. I'm sure there is still room for improvement but it will be the end of this issue.

@Grotax Grotax closed this as completed Nov 19, 2024
@Grotax Grotax unpinned this issue Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend impact Javascript/Frontend code
Projects
None yet
Development

No branches or pull requests