Skip to content

Remove limitation on toggleContent#1647

Merged
Alkarex merged 1 commit intoFreshRSS:devfrom
aledeg:back-to-previous
Sep 30, 2017
Merged

Remove limitation on toggleContent#1647
Alkarex merged 1 commit intoFreshRSS:devfrom
aledeg:back-to-previous

Conversation

@aledeg
Copy link
Copy Markdown
Member

@aledeg aledeg commented Sep 29, 2017

Before, articles were repositioned only when the new and the old article were different. When they were the same, it was just closed.
It was frustrating with long articles, especially during reading.

Now, when an article is collapsed, it is repositionned no matter the collapsing method.

This should fix #1177

Before, articles were repositioned only when the new and the old article were different. When they were the same, it was just closed.
It was frustrating with long articles, especially during reading.

Now, when an article is collapsed, it is repositionned no matter the collapsing method.

This should fix FreshRSS#1177
@aledeg aledeg added the UI 🎨 User Interfaces label Sep 29, 2017
@Frenzie
Copy link
Copy Markdown
Member

Frenzie commented Sep 29, 2017

There's a setting I've disabled because I don't like it at all. (I sometimes like to click and highlight while reading.) But nice improvement to it. 👍

@Alkarex Alkarex added this to the 1.7.1 milestone Sep 29, 2017
@Alwaysin
Copy link
Copy Markdown
Contributor

It seems I cannot update:

The update process has encountered an error: Git error: Updating 3ce2b4d..e53c10f

Any idea to why?

@Frenzie The problem is (hopefully was thanks to aledeg) here even with the setting disabled, when you use 'c' to collapse the article.

@aledeg
Copy link
Copy Markdown
Member Author

aledeg commented Sep 29, 2017

I almost committed without fixing the collapsing mecanism :D

@Alkarex
Copy link
Copy Markdown
Member

Alkarex commented Sep 29, 2017

@Alwaysin Is it an error message from the Web interface? If yes, it is likely due to some local permission problems.
Please try https://github.com/FreshRSS/FreshRSS/tree/master/cli#using-git

@Alwaysin
Copy link
Copy Markdown
Contributor

@Alkarex thank you, it works perfectly after this set of commands!
@aledeg thank you, it works perfectly after this patch!

:D :D

@Alkarex Alkarex merged commit ee83488 into FreshRSS:dev Sep 30, 2017
Alkarex added a commit that referenced this pull request Sep 30, 2017
@aledeg aledeg deleted the back-to-previous branch January 1, 2018 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

UI 🎨 User Interfaces

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants