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 smartparens-mode inserts double ' in minibuffer #16353

Merged
merged 1 commit into from May 11, 2024

Conversation

dankessler
Copy link
Contributor

smartparens-mode is sometimes activated in the minibuffer by spacemacs//conditionally-enable-smartparens-mode, specifically if we are in the minibuffer due to eval-expression (or eldoc-eval-expression). This is convenient for things like parens, but we don't want this behavior on a single quote. It looks like we tried to stop this, but for whatever reason that configuration was for minibuffer-inactive-mode rather than minibuffer-mode. The former is the major mode of the minibuffer when it is not active, whereas the latter is the major mode of the minibuffer when it is active. This has been part of spacemacs for a long time, so I've simply added configuration for minibuffer-mode and left behind the configuration for minibuffer-inactive-mode in case somebody was relying on that

smartparens-mode is sometimes activated in the minibuffer by
spacemacs//conditionally-enable-smartparens-mode, specifically if we are in the
minibuffer due to eval-expression (or eldoc-eval-expression). This is convenient
for things like parens, but we don't want this behavior on a single quote. It
looks like we *tried* to stop this, but for whatever reason that configuration
was for minibuffer-inactive-mode rather than minibuffer-mode. The former is the
major mode of the minibuffer when it is *not* active, whereas the latter is the
major mode of the minibuffer when it *is* active. This has been part of
spacemacs for a long time, so I've simply added configuration for
minibuffer-mode and left behind the configuration for minibuffer-inactive-mode
in case somebody was relying on that
@smile13241324 smile13241324 merged commit a46f4e7 into syl20bnr:develop May 11, 2024
8 checks passed
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.

None yet

2 participants