Re: [Orgmode] [PATCH] [BUG] Timestamps surrounded by @@ in HTML export
On Wed, Feb 2, 2011 at 3:18 PM, Bastien bastien.gue...@wikimedia.fr wrote: Hi Puneeth, Puneeth Chaganti puncha...@gmail.com writes: I don't really understand what is going on here, but there was one @ still appearing after this patch was applied. I am sending a patch to remove the one remaining @ symbol in the `org-html-handle-time-stamps` function. I don't know if this breaks something else and why this was left out by Jason. Better to test your own patch then ;) I did so, looks fine, I applied it. Thanks! :) I did test my patch and it worked. But these @ seemed to have been put there for some reason, I wasn't able to understand, in the quick look that I had. Anyway, thanks. -- Puneeth ___ Emacs-orgmode mailing list Please use `Reply All' to send replies to the list. Emacs-orgmode@gnu.org http://lists.gnu.org/mailman/listinfo/emacs-orgmode
Re: [Orgmode] [PATCH] [BUG] Timestamps surrounded by @@ in HTML export
Hi Puneeth, Puneeth Chaganti puncha...@gmail.com writes: I don't really understand what is going on here, but there was one @ still appearing after this patch was applied. I am sending a patch to remove the one remaining @ symbol in the `org-html-handle-time-stamps` function. I don't know if this breaks something else and why this was left out by Jason. Better to test your own patch then ;) I did so, looks fine, I applied it. Thanks! -- Bastien ___ Emacs-orgmode mailing list Please use `Reply All' to send replies to the list. Emacs-orgmode@gnu.org http://lists.gnu.org/mailman/listinfo/emacs-orgmode
Re: [Orgmode] [PATCH] [BUG] Timestamps surrounded by @@ in HTML export
Puneeth Chaganti puncha...@gmail.com writes: I don't really understand what is going on here, but there was one @ still appearing after this patch was applied. I am sending a patch to remove the one remaining @ symbol in the `org-html-handle-time-stamps` function. I don't know if this breaks something else and why this was left out by Jason. Better to test your own patch then ;) I did so, looks fine, I applied it. Thanks! :) I did test my patch and it worked. Ah okay, sorry for assuming otherwise! But these @ seemed to have been put there for some reason, I wasn't able to understand, in the quick look that I had. Anyway, thanks. It took me a while to understand this as well: when looking at org-export-as-html, you see some occurrences of @tag. These org-specific @tags are escaped and will be taken care of in the rest of the function. But org-html-handle-time-stamps happens after these @tags have been taken care of, which means that no @ is necessary in this function. Hope this helps :) -- Bastien ___ Emacs-orgmode mailing list Please use `Reply All' to send replies to the list. Emacs-orgmode@gnu.org http://lists.gnu.org/mailman/listinfo/emacs-orgmode
Re: [Orgmode] [PATCH] [BUG] Timestamps surrounded by @@ in HTML export
On Wed, Feb 2, 2011 at 3:42 PM, Bastien bastien.gue...@wikimedia.fr wrote: Puneeth Chaganti puncha...@gmail.com writes: I don't really understand what is going on here, but there was one @ still appearing after this patch was applied. I am sending a patch to remove the one remaining @ symbol in the `org-html-handle-time-stamps` function. I don't know if this breaks something else and why this was left out by Jason. Better to test your own patch then ;) I did so, looks fine, I applied it. Thanks! :) I did test my patch and it worked. Ah okay, sorry for assuming otherwise! But these @ seemed to have been put there for some reason, I wasn't able to understand, in the quick look that I had. Anyway, thanks. It took me a while to understand this as well: when looking at org-export-as-html, you see some occurrences of @tag. These org-specific @tags are escaped and will be taken care of in the rest of the function. But org-html-handle-time-stamps happens after these @tags have been taken care of, which means that no @ is necessary in this function. Hope this helps :) Thanks for explaining this. :) -- Puneeth ___ Emacs-orgmode mailing list Please use `Reply All' to send replies to the list. Emacs-orgmode@gnu.org http://lists.gnu.org/mailman/listinfo/emacs-orgmode
[Orgmode] [PATCH] [BUG] Timestamps surrounded by @@ in HTML export
I noticed in the latest Git version that each inactive timestamp has @@ before and after it in HTML exported output. Here is a patch that fixes it. diff --git a/lisp/org-html.el b/lisp/org-html.el index 6c0e5e3..2f0377b 100644 --- a/lisp/org-html.el +++ b/lisp/org-html.el @@ -2124,14 +2124,14 @@ But it has the disadvantage, that Org-mode's HTML conver (or b (setq b (substring s 0 (match-beginning 0 (setq r (concat r (substring s 0 (match-beginning 0)) - @span class=\timestamp-wrapper\ + span class=\timestamp-wrapper\ (if (match-end 1) -(format @span class=\timestamp-kwd\%s @/span +(format span class=\timestamp-kwd\%s /span (match-string 1 s))) -(format @span class=\timestamp\%s@/span +(format span class=\timestamp\%s/span (substring (org-translate-time (match-string 3 s)) 1 -1)) -@/span) +/span) s (substring s (match-end 0 ;; Line break if line started and ended with time stamp stuff (if (not r) ___ Emacs-orgmode mailing list Please use `Reply All' to send replies to the list. Emacs-orgmode@gnu.org http://lists.gnu.org/mailman/listinfo/emacs-orgmode
Re: [Orgmode] [PATCH] [BUG] Timestamps surrounded by @@ in HTML export
Jason Dunsmore emacs-orgm...@dunsmor.com writes: I noticed in the latest Git version that each inactive timestamp has @@ before and after it in HTML exported output. Here is a patch that fixes it. Applied, thanks! -- Bastien ___ Emacs-orgmode mailing list Please use `Reply All' to send replies to the list. Emacs-orgmode@gnu.org http://lists.gnu.org/mailman/listinfo/emacs-orgmode
Re: [Orgmode] [PATCH] [BUG] Timestamps surrounded by @@ in HTML export
On Tue, Feb 1, 2011 at 10:18 PM, Bastien bastien.gue...@wikimedia.fr wrote: Jason Dunsmore emacs-orgm...@dunsmor.com writes: I noticed in the latest Git version that each inactive timestamp has @@ before and after it in HTML exported output. Here is a patch that fixes it. Applied, thanks! I don't really understand what is going on here, but there was one @ still appearing after this patch was applied. I am sending a patch to remove the one remaining @ symbol in the `org-html-handle-time-stamps` function. I don't know if this breaks something else and why this was left out by Jason. diff --git a/lisp/org-html.el b/lisp/org-html.el index 2f0377b..58c65bf 100644 --- a/lisp/org-html.el +++ b/lisp/org-html.el @@ -2138,7 +2138,7 @@ But it has the disadvantage, that Org-mode's HTML conversions cannot be used. s (setq r (concat r s)) (unless (string-match \\S- (concat b s)) - (setq r (concat r @br/))) + (setq r (concat r br/))) r Thanks, Puneeth ___ Emacs-orgmode mailing list Please use `Reply All' to send replies to the list. Emacs-orgmode@gnu.org http://lists.gnu.org/mailman/listinfo/emacs-orgmode