electh / ReactFlux

A Simple but Powerful RSS Reader for Miniflux
https://reactflux.pages.dev
MIT License
215 stars 61 forks source link

[Feature] Auto refresh #127

Open hobybrenner opened 1 week ago

hobybrenner commented 1 week ago

Problem Statement

It would be nice if there was an auto refresh setting where we could have it refresh the page at intervals of our choosing. I know third party apps exist, but having it built in would be nice.

Proposed Solution

Setting for page refresh to grab updated articles.

Use Cases

when page is left open, page refreshes att intervals.

Alternative Solutions

No response

Implementation Ideas

No response

Additional Context

No response

NekoAria commented 1 week ago

I recommend not implementing this feature, for several important reasons:

  1. Following Occam's Razor principle:

    • Adding auto-refresh would increase application complexity
    • The simplest solution is often the best one
    • Browser's native refresh functionality is sufficient
  2. Technical considerations:

    • Auto-refresh can interfere with user reading experience (e.g., refreshing while reading)
    • It may cause unnecessary server load and bandwidth usage
    • Could lead to race conditions with user interactions (like marking articles as read)
    • May impact client-side state management and scroll position
    • Potential memory leaks from timer management
  3. Better alternatives:

    • Users can manually refresh when needed
    • This approach gives users more control and is less resource-intensive
  4. Maintenance burden:

    • Additional testing requirements (different intervals, browser states)
    • More error scenarios to handle (network issues, failed refreshes)
    • Increased complexity in handling background/inactive tabs

Instead of building auto-refresh, I suggest we focus on optimizing the manual refresh experience.