Ihor Radchenko writes:
> "Christopher M. Miles" writes:
>
>>> With your patch, it will become impossible to display linked images
>>> using interactive command.
>>
>> Does this code match the needs?
>
> Please check my latest reply
> (https://list.orgmode.org/87o7jpoqfl.fsf@localhost/T/#m25fbb
Ihor Radchenko writes:
> "Christopher M. Miles" writes:
>
>>> The problem here is backwards compatibility. `org-toggle-inline-images'
>>> is bound to C-c C-x C-v and people may be used to C-u C-c C-x C-v
>>> displaying linked images like
>>>
>>> [[https://orgmode.org/resources/img/org-mode-unic
Ihor Radchenko writes:
> As Max described, it might be a potential issue.
How about (1) we merge the patch, and then
(2) we add the lint warning if/when someone
has the [hypothesized] problem?
Rudy
--
"One can begin to reason only when a clear picture has been formed in
the imagination."
-- Wa
> You dropped mailing list from the CC. Was it intentional?
Whoops, no. Still figuring out an email solution in emacs.
> If I understand your concern correctly, you may simply let-bind
> `org-babel-common-header-args-
> w-values' in the tests to make sure that
> your new parameter merging code is
Mehmet Tekman writes:
>> setq unexplained-params (delete-dups unexplained-params))
>
> The help mentions that it stores the result in the list, and it looks
> like it works
>
> #+BEGIN_SRC elisp
> (let ((x '(1 2 1 2 3))) (delete-dups x) x)
> #+END_SRC
This is just implementation detail.
As a gen
Thanks for the review!
> setq unexplained-params (delete-dups unexplained-params))
The help mentions that it stores the result in the list, and it looks
like it works
#+BEGIN_SRC elisp
(let ((x '(1 2 1 2 3))) (delete-dups x) x)
#+END_SRC
>
> What we can do it make `org-babel-read' assign a text
David Masterson writes:
> I think begin.html is a good starting point. Mentioning 'texdoc latex'
> is also simple (and more LaTeX specific) and most likely to work. You
> could include a parenthetical note that, if it doesn't work, try the
> URL. After that, new users can dig through the docum
Mehmet Tekman writes:
>It's a big patch mostly, because there were no intermediate commits
>in which the org framework wouldn't be broken. Hope that's okay!
Sure it is.
> ** Problems
>
> It seems to work well for most tests, except for the "file with
> spaces.txt" which I'm not sure how
