]> git.eshelyaron.com Git - emacs.git/commitdiff
Fix wrong capture in typescript-ts-mode (bug#60167)
authorTheodor Thornhill <theo@thornhill.no>
Sat, 17 Dec 2022 19:07:59 +0000 (20:07 +0100)
committerYuan Fu <casouri@gmail.com>
Sun, 18 Dec 2022 00:30:46 +0000 (16:30 -0800)
An example of the issue could be:

    <Menu.Item>
        {({ active }) => (
          link
            ? <Link to={link}> {text}</Link>
            : <a href="#" onClick={onClick}>{text}</a>
        )}
    </Menu.Item>

Here 'link' as well as a lot of the other constructs inside of the
parenthesized expression will be font-locked with
'font-lock-variable-name-face'.  We only want to capture the
identifier.

* lisp/progmodes/typescript-ts-mode.el
(typescript-ts-mode--font-lock-settings): Make the variable capture
only capture the identifier, and not the whole expression.

lisp/progmodes/typescript-ts-mode.el

index 890d0a81348ead7b1bed17b678a5ed5dca3463f4..69616351ce303edbe36c41b8ff156c9fe74eba53 100644 (file)
@@ -231,7 +231,7 @@ Argument LANGUAGE is either `typescript' or `tsx'."
      (arguments (identifier) @font-lock-variable-name-face)
 
      (parenthesized_expression (identifier) @font-lock-variable-name-face)
-     (parenthesized_expression (_ (identifier)) @font-lock-variable-name-face))
+     (parenthesized_expression (_ (identifier) @font-lock-variable-name-face)))
 
    :language language
    :override t
@@ -316,10 +316,7 @@ Argument LANGUAGE is either `typescript' or `tsx'."
    :language language
    :feature 'escape-sequence
    :override t
-   '((escape_sequence) @font-lock-escape-face)
-
-
-   ))
+   '((escape_sequence) @font-lock-escape-face)))
 
 ;;;###autoload
 (add-to-list 'auto-mode-alist '("\\.ts\\'" . typescript-ts-mode))