Re: [evolution-patches] Re: [Evolution-hackers] Mailing lista actions bounty patch

On Mon, 2003-12-01 at 21:59, Jorge Bernal wrote:
On Mon, Dec 01, 2003 at 02:58:57PM +1100, Not Zed wrote:
> Put the "list-post" case in em-format.c:efh_format_header below all the
> address stuff, i.e. after the 'bcc' check.  Its a hell of a lot neater
> to look at.

I guess you mean in the source code (displaying order to be the same)
Yes.  It was specifically ordered that way for a reason, so you don't want to go put a big chunk of code in the middle of it.

> em-format-html can't output any buttons (i.e. <object>/s) they have to
> be done in em-format-html-display.  But since this is a lot of work
> (need to virtualise some of the header output methods and move stuff
> around), and since i think this might not be the right approach anyway,
> it should probably be done via a url link instead of an object.
> This could be done like this (it looks like a lot of work, but it isn't
> really, there's a lot of helper stuff in evolution).

So, I have to move the code to em-format-html-display anyway??
Not necessarily.

> - When we output the list-post header, we parse the address ourselves
> (rather than sending text to the text->html filter) , and output a
> mailto: uri, but add extra args, all of the list-post stuff we need
> later.  Do this using a CamelURL.  like
> address = g_strdup_printf("mailto:%s", list_post);
> url = "" NULL);
> camel_url_set_param(url, "x-list-help", list_help);
> camel_url_set_param(url, "x-list-unsubscribe", list_unsubscribe);
> etc. (but do it similar to the way the popup callbacks are done, with
> paramaterised glue)
> text = camel_url_to_string(url, 0);
> Then create the anchor and embed the image reference too, hook the image
> in using similar code to the other image code.
> Use efh_format_text_header to output the html (add
> EM_FORMAT_HTML_HEADER_HTML flag so it doesn't reformat it), dont just
> printf() it.

I don't understand at all, you mean to do this in efh_format_header or
in another place?
Well you call camel_stream_printf() to output the header text, every other bit of header text output from the same function does not do this.

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]