Yue Yi <[email protected]> writes: > Derek Chen-Becker <[email protected]> 于2026/9/28 Mon 03:04 writes： > >> Confirmed. >> >> The patch applies cleanly and all tests pass. It would be nice if there >> were a unit test as part of the patch, and although this is a bug fix >> I'm wondering if we should update docs/NEWS to clarify the behavior. > > Indeed, adding a unit test makes total sense here. I have attached the > updated patch with a new test case included for review.
Applied onto main, adding a comment to the code to clarify the reasoning. https://git.savannah.gnu.org/cgit/emacs/org-mode.git/commit/?id=99638063a Fixed. > Regarding etc/ORG-NEWS, In practice, users rarely encounter <h7> (or > lower) because the default org-export-headline-levels is 3, and with > org-html-toplevel-hlevel defaulted to 2, standard exports only reach > <h4> at most. > > If we view falling back or capping headings beyond HTML's standard as a > notable behavioral change in export output, documenting it in NEWS is > worthwhile. If we treat it purely as an internal bug fix to ensure valid > HTML markup, it might not be strictly necessary. > > I personally lean towards mentioning it in NEWS for transparency, but I > am happy to defer to the maintainers' preference. If you want, it can be: I usually avoid mentioning clear bug fixes in the news unless they can affect many users. This is a very peculiar bug that is likely encountered by a small fraction of users. So, not mentioning it in the news is OK. Our news also tend to be very long as is :) -- Ihor Radchenko // yantar92, Org mode maintainer, Learn more about Org mode at <https://orgmode.org/>. Support Org development at <https://liberapay.com/org-mode>, or support my work at <https://liberapay.com/yantar92>
