Show a message instead of hiding the posts tab when RSS is enabled - #8607
Show a message instead of hiding the posts tab when RSS is enabled#8607absidue wants to merge 2 commits into
Conversation
|
Not sure if that dim out effect on specific profiles would confuse users |
There was a problem hiding this comment.
Taken from issue
Visually grey out the toggle after the user first receives the RSS is forced message. This way, the message will only be displayed once, rather than every time.
Show toast once per tab per session
Also like discussed update the message at
https://github.com/FreeTubeApp/FreeTube/blob/development/static/locales/en-US.yaml#L128
To: “RSS is forced for profiles with 125 or more subscriptions to avoid rate limiting.”
@PikachuEXE what about this: If users doesnt explicitly toggles RSS in the settings then it will be dimmed out and forced if limit is reached in profile A but when user switches to profile B that sits below the limit then the setting will be turned off |
|
RSS forcing doenst exist anymore so this PR only needs to show the Posts tab and display a message. |
7f478b3 to
061f0c2
Compare
|
This PR is stale because it has been open 14 days with no activity. Remove stale label or comment or this will be closed in 14 days. |
Pull Request Type
Related issue
Description
Show the posts tab on the subscriptions with an explanation when "Fetch Feeds from RSS" is enabled instead of hiding it.
Screenshots
Testing
Desktop