Improve: manage feed in view within a slider#4226
Conversation
|
Known issue: It works, when slide.js is loaded after main.js I have no idea why and how to solve it. |
|
(I have a very bad Internet connectivity for the next few days, but I should be operational again next week) |
Frenzie
left a comment
Frenzie
left a comment
There was a problem hiding this comment.
lgtm besides the indentation thing
fixed |
|
Sorry for the delay. I have tested a few cases (mobile, from the list of inactive feeds, etc.) and it is a nice improvement 👍🏻 |
|
@math-GH The preview function of the full article retrieval is broken (opens in full window, and error with the content). I guess it might be related to this PR, but I have not yet investigated. |
|
I can confirm, that this PR broke this feature. Will investigate what/why happend |
|
Sorry for the delay. I wanted to test a couple of things. I will try
tonight or tomorrow.
|
I have no idea why this old comment got posted again. Probably a bug somewhere... |
3 participants
User Story:
As user I want to edit the feed name within the normal view (reader view has a bug #4224 ) without leaving the view because I found a little typo or want to config small things.
New:
the feed config opens in a side slider
Changes proposed in this pull request:
How to test the feature manually:
Pull request checklist: