On 9/10/19 9:49 PM, Michael Albinus wrote: > Eli Zaretskii writes: > >> The moment new faces are added I expect users to start complaining >> that these faces are not supported by this and that foo-mode. So mode >> authors aren't forced to implement them only in theory, IMO. > For modes in elpa, there is also the backward compatibility problem. So > they cannot feel forced to apply such new faces directly. > > Asking for similar faces for similar syntactic entities in different > modes is legitimate. IMHO. > > These new faces give mode authors a hint how to provide own customized > faces as long as necessary. And moving to the new faces will happen over > the years only. > > Best regards, Michael. Thanks everyone for your feedback. I've been quite busy the last few weeks, so sorry my lack of response. So to sum up the responses I've gotten so far: - personally would appreciate more default font-lock faces to better differentiate between things which are semanticly different, - don't personally have a need for additional default font-lock faces, but don't see anything wrong with it. - critical of the pressure and expectations adding more default-faces would put on major-mode authors. I'll be optimistic and consider that a slight majority in favour of moving forwards. Based on the discussion so far, my proposal last year[1], not to mention the recent parallell thread started by Ergus[2] we have at least these candidates: - font-lock-decorator-face - font-lock-number-face - font-lock-label-face (font-lock-label-reference-face?) - font-lock-function-name-face (font-lock-function-reference-face) - font-lock-function-declaration-face Is that a reasonable list of faces to add? Is it OK for me at this point to create a patch for this, or should that list undergo further discussion first? The advice I received last time[1] I asked about adding faces was that I would need to: - clarify usage of these modes precisely - add them to font-lock.el - document this in the proper **/*.texi files - add news of these additions to ETC/news Does that still hold true? [1] https://lists.gnu.org/archive/html/emacs-devel/2018-08/msg00258.html [2] https://lists.gnu.org/archive/html/emacs-devel/2019-09/msg00194.html -- Kind regards *Jostein Kjønigsen* jostein@kjonigsen.net 🍵 jostein@gmail.com https://jostein.kjønigsen.no