Skip to content

Commit

Permalink
workaround to make company-etags work
Browse files Browse the repository at this point in the history 
@see fxbois/web-mode#640
  • Loading branch information
Chen Bin committed Dec 8, 2015
1 parent 5fa89fd commit 2ff305c
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions site-lisp/web-mode/web-mode.el
View file
Expand Up @@ -1960,10 +1960,10 @@ another auto-completion with different ac-sources (e.g. ac-php)")

(defvar web-mode-syntax-table
(let ((table (make-syntax-table)))
(modify-syntax-entry ?- "_" table)
(modify-syntax-entry ?_ "_" table) ;; #563
;; (modify-syntax-entry ?_ "w" table)
;; (modify-syntax-entry ?- "w" table)
;; (modify-syntax-entry ?- "_" table)
;; (modify-syntax-entry ?_ "_" table) ;; #563
(modify-syntax-entry ?_ "w" table)
(modify-syntax-entry ?- "w" table)
(modify-syntax-entry ?< "." table)
(modify-syntax-entry ?> "." table)
(modify-syntax-entry ?& "." table)
Expand Down

6 comments on commit 2ff305c

@dgutov
Copy link

@dgutov dgutov commented on 2ff305c Dec 8, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

company-etags? Does etags index CSS classes?

@redguardtoo
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, the root cause is (not (company-in-string-or-comment)) in company-etags.el, web-mode doesn't matter.

@dgutov
Copy link

@dgutov dgutov commented on 2ff305c Dec 8, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suppose I can add an option company-etags-everywhere, similar to company-dabbrev-code-everywhere.

@redguardtoo
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, please, or just a list `(html-mode web-mode php-mode)

@dgutov
Copy link

@dgutov dgutov commented on 2ff305c Dec 8, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@redguardtoo
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cool, thank you.

Please sign in to comment.