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

document structure does not respect visibility for \renewcommand #3661

Open
jojo2357 opened this issue Sep 27, 2024 · 1 comment · May be fixed by #3811
Open

document structure does not respect visibility for \renewcommand #3661

jojo2357 opened this issue Sep 27, 2024 · 1 comment · May be fixed by #3811
Labels
bug Deficiencies in TeXiFy behaviour.
Milestone

Comments

@jojo2357
Copy link
Contributor

PyCharm 2024.2.2 Professional
Texify 0.9.7

File contents:

\documentclass[12pt]{article}

\newcommand{\mycmd}{Hi}
\renewcommand{\qedsymbol}{Hi}

\begin{document}

\end{document}

Goto Structure

Observe tree:
image

This is as intended.

Uncheck Show Command Definitions under the visibility settings

Observe the redefinition still present:
image

I believe this to be unintended behavior

@jojo2357 jojo2357 added bug Deficiencies in TeXiFy behaviour. untriaged Issue type still needs to be triaged or verified. labels Sep 27, 2024
@PHPirates
Copy link
Collaborator

I agree, if you want to hide definitions you probably also want to hide redefinitions

@PHPirates PHPirates removed the untriaged Issue type still needs to be triaged or verified. label Sep 28, 2024
@PHPirates PHPirates added this to the Next milestone Dec 14, 2024
@PHPirates PHPirates linked a pull request Dec 15, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Deficiencies in TeXiFy behaviour.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants