Use hierarchical Jump-to links #726
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ntrel
changed the title
ntrel
changed the title
|
Urgh, my recent related PR might have messed with that. Could you look into reconciling this? FWIW I'd love to have a small arrow next to each struct/class. When clicked, the arrow shows members in a drop down. Would that be doable? |
ntrel
force-pushed
the
jump-to-levels
branch
2 times, most recently
from
29ff9f3 to
f927058
Compare
|
Rebased on top of #740. I still need to show anchor overload links, will do tomorrow.
Should be doable, I'll investigate. Alternatively we could just show the first line of nested jump to links until expanded. |
|
This good to go? |
|
Needs a rebase since #740 was merged. |
|
rebase pliz |
No other changes except expanding $(MIDDOT).
Show non-top-level jump-to links even when inhibitQuickIndex is set. Show all jump-to links with same background color as symbol declarations.
|
@MartinNowak I've made the @andralex Should be good now, sorry for the delay. |
andralex
added a commit
that referenced
this pull request
Jan 7, 2015
Use hierarchical Jump-to links
|
... and it's up! |
|
Great! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
https://issues.dlang.org/show_bug.cgi?id=5136
Show only top-level symbols in the main module jump-to links, with remaining symbols shown in separate jump-to lists before their respective parents.
Show non-top-level jump-to links even when
inhibitQuickIndexis set.Show all jump-to links with same background color as symbol declarations.
Note: std.typecons, std.stdio and std.container.rbtree are nice modules to test this with.