Skip to content

biblatex syntax: support for concealment of additional biblatex commands#2244

Closed
krissen wants to merge 3 commits intolervag:masterfrom
krissen:master
Closed

biblatex syntax: support for concealment of additional biblatex commands#2244
krissen wants to merge 3 commits intolervag:masterfrom
krissen:master

Conversation

@krissen
Copy link
Copy Markdown
Contributor

@krissen krissen commented Nov 11, 2021

Adds support for concealment of additional biblatex commands.

(Additions to the test biblatex.tex file weren't require as the citation commands were already there. Previous to the changes suggested in this PR, several commands were not concealed. With it, all are.)

@lervag
Copy link
Copy Markdown
Owner

lervag commented Nov 11, 2021

But do we want all of them concealed? Including \bibentry?

@whisperity and @Melkster I believe you were part of the original discussion on this topic. What do you think?

@krissen
Copy link
Copy Markdown
Contributor Author

krissen commented Nov 11, 2021

But do we want all of them concealed? Including \bibentry?

Indeed, bibentry was the one I was unsure about. The rest I do believe are all used in-text, and thus ought to be concealed. I'd think that would be the logic. "Should cites be concealed? Looks at variable for cite-concealment. Yes. Alright, all citation commands used in-text will be concealed."

However, if one would like to exclude some citation commands --- instead of adding a separate variable to test (i.e. vimtex_syntax_conceal_cites or some such with a list of types of citation commands to conceal), how about simply referring to vimtex_syntax_custom_cmds for un-concealment of individual commands? (conceal: 0).

@krissen
Copy link
Copy Markdown
Contributor Author

krissen commented Nov 11, 2021

Indeed, bibentry was the one I was unsure about.

Removed bibentry from cite-concealment.

lervag added a commit that referenced this pull request Nov 12, 2021
@lervag
Copy link
Copy Markdown
Owner

lervag commented Nov 12, 2021

Thanks! I've merged. Please note: I prefer if you don't merge master into your branch unless necessary. I like to keep a logically clean Git history, and it helps when I can easily rebase a branch on top of master before I merge.

I think you are right that this makes sense, and let's instead have the discussion if someone now should oppose this.

@lervag lervag closed this Nov 12, 2021
@krissen
Copy link
Copy Markdown
Contributor Author

krissen commented Nov 12, 2021

Please note: I prefer if you don't merge master into your branch unless necessary. I like to keep a logically clean Git history, and it helps when I can easily rebase a branch on top of master before I merge.

Got it. New to contributing through PR:s. Merged master in between, thinking it was proper practice, but see why it would've been better not to. Will keep it in mind should there be any future PR:s! 👍

@lervag
Copy link
Copy Markdown
Owner

lervag commented Nov 12, 2021

I guess it is a matter of taste; I won't claim that my practise is correct or better, but it is my preferred workflow and I try to enforce it when I can :)

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.

2 participants