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

Fix issue where org-mind-map-include-text would skip parent nodes #52

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mattduck
Copy link

Hi! Thanks for working on this, I think it's great.

I know it's not maintained anymore but I'm PR'ing this just in case you or someone else finds it useful. It fixes a problem where if org-mind-map-include-text was t, text would only be included for leaf nodes that had no parent, and parent nodes would not display any text.

I've updated example-7.org and example-7.png to include the new output:

example-7

I suspect this might have been a typo. The function was returning the
point of the argument "b", rather than the start point of the first
non-matching headline. The result of this was that when
org-mind-map-include-text was enabled, text would only be displayed on
leaf nodes that had no child. For any node with a child, the text
would be skipped.

Fix the issue and update the related screenshots in the README.
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.

1 participant