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 bug with TOC creation and searching when documentclass is amsart. #262

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

Conversation

arjunkc
Copy link

@arjunkc arjunkc commented Apr 23, 2016

This was because sections that appeared as \toc\(sub\)*section in the aux file were not being parsed correctly by ReadTOC in motion.vim

I also changed it so that it would only add entries to the TOC that we
knew how to parse. This changes its previous behavior, where things
sections that appeared as \tocpart in the aux file would be added to the TOC.

Removed some substitutions for secnum and text. The title text is not as
pleasing, but it's complete.

Fix search bug in latextoc.vim when $ $ equations were present in the
title str. I changed this in s:EscapeTitle in latextoc.vim. For example

    \section{Test $asd_{12}$ $\mathcal{b}$}

This method of escaping will certainly produce duplicate matches in some
scenarios.

appeared as \toc\(sub\)*section in the aux file were not being parsed correctly by
ReadTOC in motion.vim

I also changed it so that it would only add entries to the TOC that we
knew how to parse. This changes its previous behavior, where things
sections that appeared as \tocpart in the aux file would be added to the TOC.

Removed some substitutions for secnum and text. The title text is not as
pleasing, but it's complete.

Fix search bug in latextoc.vim when `$ $` equations were present in the
title str. I changed this in s:EscapeTitle in latextoc.vim. For example

    \section{Test $asd_{12}$ $\mathcal{b}$}

This method of escaping will certainly produce duplicate matches in some
scenarios.
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