Re: [PATCH] lisp/org-table.el: Use booktabs on org-table-export

2022-05-12 Thread Ihor Radchenko
Pride Allman writes: > So basically, the export function is doing the overwrite for user options > but to reduce the number of inconsistency we can't use this patch for only > one such option. > > Now you explained it I can't think of a way to solve it without rewriting > the whole thing. Or mayb

Re: [PATCH] lisp/org-table.el: Use booktabs on org-table-export

2022-05-11 Thread Pride Allman
So basically, the export function is doing the overwrite for user options but to reduce the number of inconsistency we can't use this patch for only one such option. Now you explained it I can't think of a way to solve it without rewriting the whole thing. Or maybe I need to read the source code m

Re: [PATCH] lisp/org-table.el: Use booktabs on org-table-export

2022-05-11 Thread Ihor Radchenko
Pride Allman writes: > This is my first patch so hopefully I followed the instructions correctly, Thanks for the patch! Patches are always welcome, though I do not fully agree with the approach you used in this particular one. > While using ~org-table-export~ on a table directly even with > ~or

[PATCH] lisp/org-table.el: Use booktabs on org-table-export

2022-05-09 Thread Pride Allman
0:00:00 2001 From: Gaurav Date: Mon, 9 May 2022 18:05:52 -0400 Subject: [PATCH] lisp/org-table.el: Use booktabs on org-table-export * lisp/org-table.el (orgtbl-to-latex): Read booktabs flag from `org-latex-tables-booktabs' if `:booktabs' is not present in table `params'. Problem was w