[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Should we obsolete org-emphasis-alist? (was: [PATCH v5] org.el: Warn
From: |
Max Nikulin |
Subject: |
Re: Should we obsolete org-emphasis-alist? (was: [PATCH v5] org.el: Warning for unsupported markers in `org-set-emphasis-alist') |
Date: |
Thu, 9 Feb 2023 19:11:29 +0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.7.1 |
On 07/02/2023 17:47, Ihor Radchenko wrote:
Upon looking closer, `org-set-emph-re' is not affected by the value of
`org-emphasis-alist' with a single exception when the value is nil.
You are right, it is result of the following commit:
9fb2e047d 2016-12-08 09:44:26 +0100 Nicolas Goaziou: Split `org-emph-re'
and `org-verbatim-re'
I think that we should simply obsolete `org-emphasis-alist' and
introduce individual faces for emphasis.
I agree with your conclusions and proposal.
If you are close to merging fontification based on org-element then we
may just discard this patch. Otherwise it may be merged with minimal
modifications:
- Do not add the watcher
- Instead of `org-set-emph-re' use a lambda calling new
`org-emphasis-alist--check-value` as the :set attribute of `defcustom'.
- Fix description of verbatim tag.