Re: Another HTML Export Problem

all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed

* Re: Another HTML Export Problem
@ 2011-03-16 11:51 David Maus
  2011-03-17 14:34 ` Eric S Fraga
  0 siblings, 1 reply; 7+ messages in thread
From: David Maus @ 2011-03-16 11:51 UTC (permalink / raw)
  To: bzg; +Cc: emacs-orgmode

Hi,

Bastien's patch looks good and/but introduces a regression: The commit
that started all this trobule --
163cd58ffd6461c98a96b1b63a3cf082b2825a52 -- fixed a problem with
exporting links with a description that contained an ISO Date.  The
current problem can be located in `org-html-do-expand' which uses the
regexp "@&lt;\\([^&]*\\)&gt;" to remove protection from elements.
Obviously this function was never meant to deal with links :|

I think we should accept the regression, restore the old behaviour and
find a better solution for the problem I tried to solve in the first
place.

Best,
  -- David

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: Re: Another HTML Export Problem
  2011-03-16 11:51 Another HTML Export Problem David Maus
@ 2011-03-17 14:34 ` Eric S Fraga
  2011-03-20 11:21   ` David Maus
  0 siblings, 1 reply; 7+ messages in thread
From: Eric S Fraga @ 2011-03-17 14:34 UTC (permalink / raw)
  To: David Maus; +Cc: bzg, emacs-orgmode

David Maus <maus.david@gmail.com> writes:

> Hi,
>
> Bastien's patch looks good and/but introduces a regression: The commit
> that started all this trobule --
> 163cd58ffd6461c98a96b1b63a3cf082b2825a52 -- fixed a problem with
> exporting links with a description that contained an ISO Date.  The
> current problem can be located in `org-html-do-expand' which uses the
> regexp "@&lt;\\([^&]*\\)&gt;" to remove protection from elements.
> Obviously this function was never meant to deal with links :|
>
> I think we should accept the regression, restore the old behaviour and
> find a better solution for the problem I tried to solve in the first
> place.
>
> Best,
>   -- David
>

I don't know if this is the same problem but exporting, to HTML, the
following

#+begin_src org
[[http://petition.stopsoftwarepatents.eu/831004564935/][http://petition.stopsoftwarepatents.eu/banner/831004564935/ssp-362-60.gif]]
#+end_src

generates the following HTML:

--8<---------------cut here---------------start------------->8---
<a href="http://petition.stopsoftwarepatents.eu/831004564935/">@<img src="http://petition.stopsoftwarepatents.eu/banner/831004564935/ssp-362-60.gif"/></a>
--8<---------------cut here---------------end--------------->8---

Note the spurious "@" introduced before the =<img= tag.  This only seems
to be affecting my links to images, as far as I can tell.

Thanks,
eric

-- 
: Eric S Fraga (GnuPG: 0xC89193D8FFFCF67D) in Emacs 24.0.50.1
: using Org-mode version 7.5 (release_7.5.74.g294dc)

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: Re: Another HTML Export Problem
  2011-03-17 14:34 ` Eric S Fraga
@ 2011-03-20 11:21   ` David Maus
  2011-03-20 19:44     ` Eric S Fraga
  2011-03-21  1:46     ` Bernt Hansen
  0 siblings, 2 replies; 7+ messages in thread
From: David Maus @ 2011-03-20 11:21 UTC (permalink / raw)
  To: Eric S Fraga; +Cc: David Maus, emacs-orgmode, bzg

[-- Attachment #1: Type: text/plain, Size: 309 bytes --]

At Thu, 17 Mar 2011 14:34:52 +0000,
Eric S Fraga wrote:
>
> I don't know if this is the same problem but exporting, to HTML, the

Yes, this was a symptom of the broken HTML export. Should be fixed now.

Best,
  -- David
--
OpenPGP... 0x99ADB83B5A4478E6
Jabber.... dmjena@jabber.org
Email..... dmaus@ictsoc.de

[-- Attachment #2: Type: application/pgp-signature, Size: 230 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: Re: Another HTML Export Problem
  2011-03-20 11:21   ` David Maus
@ 2011-03-20 19:44     ` Eric S Fraga
  2011-03-21  1:46     ` Bernt Hansen
  1 sibling, 0 replies; 7+ messages in thread
From: Eric S Fraga @ 2011-03-20 19:44 UTC (permalink / raw)
  To: David Maus; +Cc: emacs-orgmode

David Maus <maus.david@gmail.com> writes:

> At Thu, 17 Mar 2011 14:34:52 +0000,
> Eric S Fraga wrote:
>>
>> I don't know if this is the same problem but exporting, to HTML, the
>
> Yes, this was a symptom of the broken HTML export. Should be fixed
> now.

It is indeed.  Many thanks.

-- 
: Eric S Fraga (GnuPG: 0xC89193D8FFFCF67D) in Emacs 24.0.50.1
: using Org-mode version 7.5 (release_7.5.90.g1fb3.dirty)

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: Another HTML Export Problem
  2011-03-20 11:21   ` David Maus
  2011-03-20 19:44     ` Eric S Fraga
@ 2011-03-21  1:46     ` Bernt Hansen
  2011-03-21  5:46       ` David Maus
  1 sibling, 1 reply; 7+ messages in thread
From: Bernt Hansen @ 2011-03-21  1:46 UTC (permalink / raw)
  To: David Maus; +Cc: bzg, emacs-orgmode

David Maus <maus.david@gmail.com> writes:

> At Thu, 17 Mar 2011 14:34:52 +0000,
> Eric S Fraga wrote:
>>
>> I don't know if this is the same problem but exporting, to HTML, the
>
> Yes, this was a symptom of the broken HTML export. Should be fixed now.

Hi David,

I think there is yet another HTML export issue with the current
development code.

--8<---------------cut here---------------start------------->8---
* Test
[2011-03-20 Sun]
[2011-03-20 Sun]
org.el
--8<---------------cut here---------------end--------------->8---

exported here: http://www.norang.ca/tmp/org-scratch.html

exports to HTML as

--8<---------------cut here---------------start------------->8---
...
<body>
<div id="content">

<h1 class="title">org-scratch</h1>

<div id="table-of-contents">
<h2>Table of Contents</h2>
<div id="text-table-of-contents">
<ul>
<li><a href="#sec-1">1 Test </a></li>
</ul>
</div>
</div>

<div id="outline-container-1" class="outline-2">
<h2 id="sec-1"><span class="section-number-2">1</span> Test </h2>
<div class="outline-text-2" id="text-1">

<p> <span class="timestamp-wrapper"> <span class="timestamp">2011-03-20 Sun</span></span>&lt;br/&gt;
 <span class="timestamp-wrapper"> <span class="timestamp">2011-03-20 Sun</span></span>&lt;br/&gt;
org.el
</p></div>
</div>
<div id="postamble">
<p class="date">Date: 2011-03-20 21:31:20 EDT</p>
<p class="author">Author: Bernt Hansen</p>
<p class="creator">Org version 7.5 with Emacs version 23</p>
<a href="http://validator.w3.org/check?uri=referer">Validate XHTML 1.0</a>
</div>
</div>
</body>
</html>
--8<---------------cut here---------------end--------------->8---

For the timestamps

,----
| <div class="outline-text-2" id="text-1">
| 
| <p> <span class="timestamp-wrapper"> <span class="timestamp">2011-03-20 Sun</span></span>&lt;br/&gt;
|  <span class="timestamp-wrapper"> <span class="timestamp">2011-03-20 Sun</span></span>&lt;br/&gt;
| org.el
| </p></div>
`----

The timestamps have the < and > brackets of <br/> converted to &lt; and
&gt; which is incorrect.

I ran bisect and identified the following commit

6a369c26d3f936bc71cba9d7148dcecf1b2c9677 is the first bad commit
commit 6a369c26d3f936bc71cba9d7148dcecf1b2c9677
Author: David Maus <dmaus@ictsoc.de>
Date:   Sun Mar 13 20:27:12 2011 +0100

    Expand char entities after creating markup for links and timestamps
    
    * org-html.el (org-html-make-link, org-html-handle-links): Protect
    generated XHTML elements.
    (org-export-as-html): Expand character entities after creating markup
    for links and timestamps.
    
    This fixes a problem with exporting active timestamps, reported by
    Daniel Clemente <n142857@gmail.com>.

:040000 040000 1508340e77dca86766af48df2eadf747070f89ce ca668c398391b307e49113fb592fb68195640896 M	lisp

Regards,
-- 
Bernt

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: Another HTML Export Problem
  2011-03-21  1:46     ` Bernt Hansen
@ 2011-03-21  5:46       ` David Maus
  2011-03-21 12:49         ` Bernt Hansen
  0 siblings, 1 reply; 7+ messages in thread
From: David Maus @ 2011-03-21  5:46 UTC (permalink / raw)
  To: Bernt Hansen; +Cc: David Maus, emacs-orgmode, bzg

[-- Attachment #1: Type: text/plain, Size: 576 bytes --]

Hi Bernt,

At Sun, 20 Mar 2011 21:46:19 -0400,
Bernt Hansen wrote:
>
> David Maus <maus.david@gmail.com> writes:
>
> > At Thu, 17 Mar 2011 14:34:52 +0000,
> > Eric S Fraga wrote:
> >>
> >> I don't know if this is the same problem but exporting, to HTML, the
> >
> > Yes, this was a symptom of the broken HTML export. Should be fixed now.
>
> I think there is yet another HTML export issue with the current
> development code.

Indeed. This should be fixed now as well.

Best,
  -- David
--
OpenPGP... 0x99ADB83B5A4478E6
Jabber.... dmjena@jabber.org
Email..... dmaus@ictsoc.de

[-- Attachment #2: Type: application/pgp-signature, Size: 230 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: Another HTML Export Problem
  2011-03-21  5:46       ` David Maus
@ 2011-03-21 12:49         ` Bernt Hansen
  0 siblings, 0 replies; 7+ messages in thread
From: Bernt Hansen @ 2011-03-21 12:49 UTC (permalink / raw)
  To: David Maus; +Cc: bzg, emacs-orgmode

David Maus <maus.david@gmail.com> writes:

> At Sun, 20 Mar 2011 21:46:19 -0400,
> Bernt Hansen wrote:
>>
>> I think there is yet another HTML export issue with the current
>> development code.
>
> Indeed. This should be fixed now as well.

Hi David,

Thanks for the quick fix!

Regards,
-- 
Bernt

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2011-03-21 12:49 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-03-16 11:51 Another HTML Export Problem David Maus
2011-03-17 14:34 ` Eric S Fraga
2011-03-20 11:21   ` David Maus
2011-03-20 19:44     ` Eric S Fraga
2011-03-21  1:46     ` Bernt Hansen
2011-03-21  5:46       ` David Maus
2011-03-21 12:49         ` Bernt Hansen

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.