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

Remove option to disable prev/next #2333

Open
wants to merge 2 commits into
base: development
Choose a base branch
from

Conversation

joshuaadickerson
Copy link
Contributor

@emanuele45
Copy link
Contributor

Judging from the discussion, I feel we can wait a moment and see if we can come up with a somehow better solution about the prev/next thingy.

@Spuds
Copy link
Contributor

Spuds commented Jan 2, 2016

should still 🍒 the bug out this this PR both here and 1.0.7

@live627
Copy link
Contributor

live627 commented Sep 7, 2016

???

@emanuele45
Copy link
Contributor

IIRC the topicPointer issue is fixed... unless it is not.

The prev/next: the PR is here because the issue is still open: the commit here enables it by default and removes the option to turn it off. The general idea proposed on the forum was either leave it as it is or remove entirely the feature and put it into an addon.

Since there were points on both the sides, I would... keep it on hold for now and think about it for 2.0 having some better way to add it back without having to deal with code edits.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants