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

Added setting for self-post collapse #503

Merged
merged 1 commit into from
May 24, 2017

Conversation

Clubfan22
Copy link
Contributor

It was requested on reddit: https://www.reddit.com/r/RedReader/comments/6bv9ma/an_option_to_turn_off_collapsing_of_selfposts_on/

For now, one can choose between collapsing the text or doing nothing, but it can be (almost) trivially extended to support more actions.

One might also want to refactor the self-post code in the future to be treated the same as a comment in some cases.

It was requested on reddit: https://www.reddit.com/r/RedReader/comments/6bv9ma/an_option_to_turn_off_collapsing_of_selfposts_on/

For now, one can choose between collapsing the text or doing nothing, but it can be (almost) trivially extended to support more actions.

One might also want to refactor the self-post code in the future to be treated the same as a comment in some cases.
@QuantumBadger
Copy link
Owner

Great, thank you! Merging in now.

One might also want to refactor the self-post code in the future to be treated the same as a comment in some cases.

Possibly, although there are a few differences (e.g. long-click menu actions, swipe actions, etc).

@QuantumBadger QuantumBadger merged commit 2698b0d into QuantumBadger:master May 24, 2017
@QuantumBadger
Copy link
Owner

Changelog updated: 0915f27

@Clubfan22 Clubfan22 deleted the self-post-setting branch October 25, 2017 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants