* [BUG] External unicode links without a description in ox-html
@ 2016-07-20 14:44 Arun Isaac
2016-07-20 20:04 ` Nicolas Goaziou
0 siblings, 1 reply; 6+ messages in thread
From: Arun Isaac @ 2016-07-20 14:44 UTC (permalink / raw)
To: org mode
[-- Attachment #1: Type: text/plain, Size: 335 bytes --]
For external links without a description, `org-html-link' resuses the
`path' as the inner text of the exported <a> element. However, the
`path' variable contains the URL encoded version of the `raw-path'. It
would be more appropriate to put the actual unescaped unicode URL in the
inner text of the <a> element.
Regards,
Arun Isaac.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 472 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [BUG] External unicode links without a description in ox-html
2016-07-20 14:44 [BUG] External unicode links without a description in ox-html Arun Isaac
@ 2016-07-20 20:04 ` Nicolas Goaziou
2016-07-23 15:34 ` Michael Brand
0 siblings, 1 reply; 6+ messages in thread
From: Nicolas Goaziou @ 2016-07-20 20:04 UTC (permalink / raw)
To: Arun Isaac; +Cc: org mode
Hello,
Arun Isaac <arunisaac@systemreboot.net> writes:
> For external links without a description, `org-html-link' resuses the
> `path' as the inner text of the exported <a> element. However, the
> `path' variable contains the URL encoded version of the `raw-path'. It
> would be more appropriate to put the actual unescaped unicode URL in the
> inner text of the <a> element.
Done. Thanks.
Regards,
--
Nicolas Goaziou
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [BUG] External unicode links without a description in ox-html
2016-07-20 20:04 ` Nicolas Goaziou
@ 2016-07-23 15:34 ` Michael Brand
2016-07-25 12:52 ` Nicolas Goaziou
0 siblings, 1 reply; 6+ messages in thread
From: Michael Brand @ 2016-07-23 15:34 UTC (permalink / raw)
To: org mode
Hi Nicolas
There seems to be a related issue with an inconsistency between HTML
and other export formats in using org-link-unescape for the link
_destination_ part: With the Org file
1) https://duckduckgo.com/?q=Org+mode+%252B+Worg
2) https://duckduckgo.com/?q=Org+mode+%2B+Worg
org-open-at-point on link 1) opens a web browser with the search field
filled with "Org mode + Worg" as expected by me. The same happens when
using link 1) of the HTML export. But when exporting to PDF (via
LaTeX), ODT or ASCII (browse-url-at-point) I have to use link 2) to
get the same result. I think one should be able to consistently use
link 1) for all export formats.
Michael
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [BUG] External unicode links without a description in ox-html
2016-07-23 15:34 ` Michael Brand
@ 2016-07-25 12:52 ` Nicolas Goaziou
2016-07-25 20:33 ` Michael Brand
0 siblings, 1 reply; 6+ messages in thread
From: Nicolas Goaziou @ 2016-07-25 12:52 UTC (permalink / raw)
To: Michael Brand; +Cc: org mode
Hello,
Michael Brand <michael.ch.brand@gmail.com> writes:
> There seems to be a related issue with an inconsistency between HTML
> and other export formats in using org-link-unescape for the link
> _destination_ part: With the Org file
>
> 1) https://duckduckgo.com/?q=Org+mode+%252B+Worg
> 2) https://duckduckgo.com/?q=Org+mode+%2B+Worg
>
> org-open-at-point on link 1) opens a web browser with the search field
> filled with "Org mode + Worg" as expected by me.
This looks like an error to me.
If I type https://duckduckgo.com/?q=Org+mode+%252B+Worg in my browser,
I get
"Org mode %2B Worg"
as the search string. It should be the same when opening the link from
an Org document. These URI are /not/ equivalent.
> The same happens when using link 1) of the HTML export. But when
> exporting to PDF (via LaTeX), ODT or ASCII (browse-url-at-point)
> I have to use link 2) to get the same result. I think one should be
> able to consistently use link 1) for all export formats.
It looks as we're trying to paper over an Org problem here, which is the
redundant link escaping that happens when calling `org-insert-link' (C-c
C-l).
AFAICT, there are two reasons for Org to escape a link: when the link
contains either "]]" or multiple consecutive spaces. The former
obviously breaks Org link syntax. The latter doesn't survive a call to
`fill-paragraph'.
Alas, Org handles it the wrong way, by using a mechanism that cannot be
properly undone; you cannot possibly know how many times the desired URI
has been encoded, if at all. Moreover, this mechanism isn't user
friendly, i.e., you cannot reasonably ask a user to encode an URI on the
fly when jolting notes.
I can see two ways out:
1. Do not escape anything.
This prevent any link with a description to contain either "]]" or
multiple spaces, but these requirements are so uncommon we probably
shouldn't bother.
2. Use a different internal escape mechanism.
By providing our own simple escape mechanism, e.g., \]\], we can
solve the issues raised above.
In any case, Org should not create something as
https://duckduckgo.com/?q=Org+mode+%252B+Worg
if the real URI is
https://duckduckgo.com/?q=Org+mode+%2B+Worg
WDYT?
Regards,
--
Nicolas Goaziou
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [BUG] External unicode links without a description in ox-html
2016-07-25 12:52 ` Nicolas Goaziou
@ 2016-07-25 20:33 ` Michael Brand
2016-07-26 19:05 ` Nicolas Goaziou
0 siblings, 1 reply; 6+ messages in thread
From: Michael Brand @ 2016-07-25 20:33 UTC (permalink / raw)
To: org mode
Hi Nicolas
Your suggestions are so convincing in going so far, I hope I
understand them right. If yes it is just thinking in terms of "[[",
"][" and "]]" instead of single brackets that I got used to with the
current escaping and unescaping in Org.
On Mon, Jul 25, 2016 at 2:52 PM, Nicolas Goaziou <mail@nicolasgoaziou.fr> wrote:
> Michael Brand <michael.ch.brand@gmail.com> writes:
>> There seems to be a related issue with an inconsistency between HTML
>> and other export formats in using org-link-unescape for the link
>> _destination_ part: With the Org file
>>
>> 1) https://duckduckgo.com/?q=Org+mode+%252B+Worg
>> 2) https://duckduckgo.com/?q=Org+mode+%2B+Worg
>>
>> org-open-at-point on link 1) opens a web browser with the search field
>> filled with "Org mode + Worg" as expected by me.
>
> This looks like an error to me.
>
> If I type https://duckduckgo.com/?q=Org+mode+%252B+Worg in my browser,
> I get
>
> "Org mode %2B Worg"
>
> as the search string. It should be the same when opening the link from
> an Org document. These URI are /not/ equivalent.
>
>> The same happens when using link 1) of the HTML export. But when
>> exporting to PDF (via LaTeX), ODT or ASCII (browse-url-at-point)
>> I have to use link 2) to get the same result. I think one should be
>> able to consistently use link 1) for all export formats.
>
> It looks as we're trying to paper over an Org problem here, which is the
> redundant link escaping that happens when calling `org-insert-link' (C-c
> C-l).
>
> AFAICT, there are two reasons for Org to escape a link: when the link
> contains either "]]" or multiple consecutive spaces. The former
> obviously breaks Org link syntax. The latter doesn't survive a call to
> `fill-paragraph'.
>
> Alas, Org handles it the wrong way, by using a mechanism that cannot be
> properly undone; you cannot possibly know how many times the desired URI
> has been encoded, if at all. Moreover, this mechanism isn't user
> friendly, i.e., you cannot reasonably ask a user to encode an URI on the
> fly when jolting notes.
I agree.
> I can see two ways out:
>
> 1. Do not escape anything.
>
> This prevent any link with a description to contain either "]]" or
... a single bracket at the border or a link destination part to
contain "][" or "]]" or a single bracket at the border or ...
> multiple spaces, but these requirements are so uncommon we probably
> shouldn't bother.
I never had such links and don't bother. If I am right these could
even be tweaked manually with %20, %5B and %5D to get working.
I can't tell for everyone but would happily adapt the escaped ones of
all my existing Org links accordingly if such a change happens in Org.
> 2. Use a different internal escape mechanism.
>
> By providing our own simple escape mechanism, e.g., \]\], we can
> solve the issues raised above.
In my opinion not necessary. Can be added later if really needed
anyway.
> In any case, Org should not create something as
>
> https://duckduckgo.com/?q=Org+mode+%252B+Worg
>
> if the real URI is
>
> https://duckduckgo.com/?q=Org+mode+%2B+Worg
>
> WDYT?
I agree.
Do I understand right that not escaping and unescaping would allow
: https://duckduckgo.com/?q=[dest]dest
: [[https://duckduckgo.com/?q=[dest]dest]]
: [[https://duckduckgo.com/?q=[dest]dest][desc[desc]desc]]
etc. and even the same with link abbreviations instead of http(s)?
Michael
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [BUG] External unicode links without a description in ox-html
2016-07-25 20:33 ` Michael Brand
@ 2016-07-26 19:05 ` Nicolas Goaziou
0 siblings, 0 replies; 6+ messages in thread
From: Nicolas Goaziou @ 2016-07-26 19:05 UTC (permalink / raw)
To: Michael Brand; +Cc: org mode
Hello,
Michael Brand <michael.ch.brand@gmail.com> writes:
>> This prevent any link with a description to contain either "]]" or
>
> ... a single bracket at the border or a link destination part to
> contain "][" or "]]" or a single bracket at the border or ...
Correct.
>> multiple spaces, but these requirements are so uncommon we probably
>> shouldn't bother.
>
> I never had such links and don't bother. If I am right these could
> even be tweaked manually with %20, %5B and %5D to get working.
Not really, because Org would no longer un-escape the URI. This could
work for links opened with a browser, but not for other types (e.g.,
a target).
> Do I understand right that not escaping and unescaping would allow
>
> : https://duckduckgo.com/?q=[dest]dest
This one is already possible, isn't it? Also, I suggest to change
brackets links, not plain links.
> : [[https://duckduckgo.com/?q=[dest]dest]]
> : [[https://duckduckgo.com/?q=[dest]dest][desc[desc]desc]]
Yes, those would become valid. The following regexp could be used as
a replacement for `org-bracket-link-regexp'
"\\[\\[[^]+?\\(\\]\\[[^]]\\)?\\]\\]"
or, with groups,
"\\[\\[\\([^]+?\\)\\(?:\\]\\[\\([^]+?\\)\\)?\\]\\]"
Regards,
--
Nicolas Goaziou
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-07-26 19:05 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-07-20 14:44 [BUG] External unicode links without a description in ox-html Arun Isaac
2016-07-20 20:04 ` Nicolas Goaziou
2016-07-23 15:34 ` Michael Brand
2016-07-25 12:52 ` Nicolas Goaziou
2016-07-25 20:33 ` Michael Brand
2016-07-26 19:05 ` Nicolas Goaziou
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).