On Tue, Dec 23, 2008 at 8:57 AM, Sebastien Pouliot <
[email protected]> wrote:

> On Sun, 2008-12-21 at 22:14 +0800, Cedric Vivier wrote:
> > This patch improves html report a little by :
> > - having a colored border related to the severity of a defect
>
> Are those the same colors that we use on the console ? (the default
> scheme)
>

Yes, except for Severity.Low (green border) and Severity.Audit (blue border)
since we do not have colors for them the console - should we?



>
> > - presenting a tooltip with the rule name and target when the user
> > hovers a rule with the cursor.
> > - displaying "X defects found in:" instead of "Found in:"
>
> Better use "defect(s)" since there could be a single one


Ok.


>
> > - displaying Problem and Solution together... which enables:
>
> I'm a bit less sure about this but lets try it on HEAD for a while...
>
> > - adding a discrete link to go back to the rule description from any
> > defect.
>
> That's a good idea as the list can get quite long for some rules.
>

I'll commit then.

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Gendarme" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.com/group/gendarme?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to