Jens Lechtenboerger <lech...@wi.uni-muenster.de> writes:
> I do not know why the CDATA lines exist. I don’t see a reason to > keep them (patch 0001), but that might be a lack of understanding on > my part. I'll cover this in my reply to your follow-up. > Patch 0003 is about whitespace fixes. > > Patches 0002, 0004, 0005 change defconst styling. I don’t have a > strong opinion here. However, if they are changed now, what about > turning them into defcustoms? Then each of us would be entitled to > their own opinion ;) > > The docstring for org-html-head-include-default-style says that > org-html-style-default (a defconst proposed to be changed here) > should not be changed. Why not? The defconst is left a defsconst because I don't really know why it is one, and when I don't know why something is the way it is, I try to leave it alone. In my own config I actually overwrite it, but oh well. I'm guessing this is just a case of "User beware, this contains important stuff to have Org look reasonable. Don't remove that." Perhaps this should be a defcustom? It would be good to hear from someone else on this. If this ends up being a hold up though I'd rather resolve this separately. -- Timothy.
