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.

Sign up for GitHub

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

Jump to bottom

Support combined use of sectanchors and sectlinks #1806

Closed
pniederw opened this issue Jun 27, 2016 · 1 comment
Closed

Support combined use of sectanchors and sectlinks #1806

pniederw opened this issue Jun 27, 2016 · 1 comment
Assignees
@mojavelinux
Labels
improvement
Milestone
v1.5.5

Comments

@pniederw
Copy link

pniederw commented Jun 27, 2016 •
edited by mojavelinux

I like sectanchors for its visual feedback. I like sectlinks for the ease of clicking section titles. Can we have both? (Currently, adding both attributes doesn't have the desired effect.)
@mojavelinux mojavelinux added this to the v1.5.5 milestone Jun 30, 2016
@mojavelinux mojavelinux self-assigned this Jun 30, 2016
@mojavelinux
Copy link
Member

You are correct that these attributes are currently mutually exclusive. I don't see the harm in taking away that restriction. It's a minor change.

@mojavelinux mojavelinux changed the title Support combined use of :sectanchors and :sectlinks Support combined use of sectanchors and sectlinks Oct 3, 2016
mojavelinux added a commit to mojavelinux/asciidoctor that referenced this issue Oct 4, 2016
@mojavelinux
resolves asciidoctor#1806 allow sectanchors & sectlinks to be used in…
b355518 
… tandem

- allow sectanchors & sectlinks to be used in tandem in the HTML5 converter
@mojavelinux mojavelinux removed the next label Nov 26, 2016
ggrossetie pushed a commit to ggrossetie/asciidoctor that referenced this issue Mar 11, 2017
@mojavelinux @ggrossetie
resolves asciidoctor#1806 allow sectanchors & sectlinks to be used in…
b742460 
… tandem (PR asciidoctor#1886)

- allow sectanchors & sectlinks to be used in tandem in the HTML5 converter
@mojavelinux mojavelinux mentioned this issue Apr 19, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mojavelinux mojavelinux

Labels
improvement
Projects
None yet
Milestone
v1.5.5
Development

No branches or pull requests
2 participants
@pniederw @mojavelinux