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

Have \includeonly in the default setting of g:vimtex_include_indicators #2519

Closed
krishnakumarg1984 opened this issue Oct 17, 2022 · 4 comments
Labels

Comments

@krishnakumarg1984
Copy link
Contributor

Description

Currently, g:vimtex_include_indicators contains the default value of ['input', 'include']. It is perhaps worthwhile to additionally have \includeonly as the default.

https://latexref.xyz/_005cinclude-_0026-_005cincludeonly.html

When authoring chapters of a large book, maybe we want to see only some related chapters, and \includeonly is very handy.

Steps to reproduce

No response

Expected behavior

No response

Actual behavior

No response

Do you use a latexmkrc file?

N/A

VimtexInfo

N/A
@lervag
Copy link
Owner

lervag commented Oct 17, 2022

Sounds reasonable!

@lervag lervag closed this as completed Oct 17, 2022
@AsbjornPreuss
Copy link
Contributor

This seems to add some problems with the TODO implementation, where all TODOs will be shown, but the TODOs in files, that are \includeonly, will show up twice. Is this the intended functionality? Is there someway to avoid it?
vimtex_TODO_includeonly

@lervag
Copy link
Owner

lervag commented Nov 19, 2024

You may be right. Can you open a new issue with a minimal example?

@lervag
Copy link
Owner

lervag commented Nov 19, 2024

Hmm... considering this more closely, I think you may be right that it was wrong to add \includeonly to the indicators option. This command does not include anything, it is only used as a "guard" for the \include command.

Still, it would be nice with a new issue that references this one and that gives a simple example.

lervag added a commit that referenced this issue Nov 21, 2024
This is basically a revert of 60c8a7f that was added due to #2519. It
seems correctly pointed out in #3044 that the change made at the time
was not correct.

refer: #3044, #3045
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants