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 rolling links #532

Closed
hakimel opened this issue Jul 21, 2013 · 2 comments
Closed

Remove rolling links #532

hakimel opened this issue Jul 21, 2013 · 2 comments

Comments

@hakimel
Copy link
Owner

hakimel commented Jul 21, 2013

They're buggy when the link text is large or breaks into multiple lines. I want the default style of the framework to be (a bit) less flashy and more functional.

hakimel added a commit that referenced this issue Jul 21, 2013
@hakimel hakimel closed this as completed Jul 21, 2013
@dandv
Copy link
Contributor

dandv commented Jul 21, 2013

Sad to see the rolling links go. If the more common case is to have single-line links, perhaps it would serve more users to keep them and document the issue with multi-line links?

@hakimel
Copy link
Owner Author

hakimel commented Jul 26, 2013

@dandv Feedback heard. Though I really don't like having an unstable feature in there, immediately breaking backwards compatibility might not be the right move. Rolling links are back in but turned off by default (deprecated) and the long term plan is still to remove them entirely.

GerHobbelt added a commit to GerHobbelt/reveal.js that referenced this issue Aug 16, 2013
enovajuan pushed a commit to enovajuan/reveal.js that referenced this issue Dec 2, 2014
enovajuan pushed a commit to enovajuan/reveal.js that referenced this issue Dec 2, 2014
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

No branches or pull requests

2 participants