[privoxy-devel] [ ijbswa-Bugs-549871 ] Templatize "standard actions" buttons

[<no...@so...>]

Bugs item #549871, was opened at 2002-04-28 19:09
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=111118&aid=549871&group_id=11118

Category: funct: cgi
Group: version 2.9.x
Status: Open
Resolution: None
Priority: 9
Submitted By: Jon Foster (jongfoster)
Assigned to: Jon Foster (jongfoster)
>Summary: Templatize "standard actions" buttons

Initial Comment:
Hi,

This bug report is just to stop duplication of effort.
Ignore if you're not working on the code right now.


The edit-actions-list CGI doesn't use a template for
the HTML used for the "Set to Cautious" and "Set to
Advanced" buttons.  It should.

I have this change in my local sandbox, but I won't have
time to commit it until Monday evening - I need to
separate it from another change I'm working on.

(I've also changed the design of the edit actions UI as
a result of jzw's comments, but this needs some
discussion - I'll submit it as a patch for comments,
when I've done the code change mentioned above.)

Jon


----------------------------------------------------------------------

>Comment By: Jon Foster (jongfoster)
Date: 2002-04-30 19:14

Message:
Logged In: YES 
user_id=199289

Hi,

> Yes -- I had only added a FIXME to this effect
> because I thought that the extra template handling
> might be too risky to do before 3.0.

Yep, I agree that that was probably best.

> Just note that there are many more (old & new)
> spots in the code where we use hardcoded HTML
> that ought to be replaced with line templates:

> I don't feel very strongly about whether to fix
> all these before 3.0, 

I'm not too fussed either way, but leaving it alone
is less risky.

However, in this specific case, ...

>> (I've also changed the design of the edit actions UI
>> as a result of jzw's comments,

... I was going to edit the template that was hard-coded
into the C function.  At that point it becomes easier
to do it properly: templatize it once, and then all the UI
changes are low-risk.

Blue skies,

Jon


----------------------------------------------------------------------

Comment By: Andreas Oesterhelt (oes)
Date: 2002-04-30 13:59

Message:
Logged In: YES 
user_id=78811

Hi Jon,

> The edit-actions-list CGI doesn't use a template for
> the HTML used for the "Set to Cautious" and "Set to
> Advanced" buttons. It should.

Yes -- I had only added a FIXME to this effect
because I thought that the extra template handling
might be too risky to do before 3.0. But you're 
propably right and we should do it.

Just note that there are many more (old & new)
spots in the code where we use hardcoded HTML
that ought to be replaced with line templates:

actions.c:

 - line formatting in action_to_html and
   current_action_to_html (maybe both not
   worth templatizing)

cgi.c:

 - menu items in make_menu
 - table in dump_map (not worth it)

cgisimple.c:

 - extra parameters case in default_cgi (just removed)
 - "file open error" case in cgi_show_status (should
   be a map_conditional)
 - View / Edit button generation, in cgi_show_status
 - actions file name headers (with buttons), cells for 
   matching actions and "none match" cell in
   cgi_show_url_info

filters.c:

 - trust and trust-info list items in trust_url

loadcfg.c: (not worth it, imho)

 - errors while reading config
 - config options in savearg

I don't feel very strongly about whether to fix
all these before 3.0, but if you'd prefer it done,
just tell me which areas of the code are currently
"locked" by you, so I can start on the rest.

> (I've also changed the design of the edit actions UI as
> a result of jzw's comments,

Cool. I'm curious!

--Andreas


----------------------------------------------------------------------

You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=111118&aid=549871&group_id=11118