On Fri, Jul 10, 2020 at 9:50 PM Sergey Ponomarev <stok...@gmail.com> wrote: > Divide table to thead, tbody and tfoot as it should be in DOM. > Add row hover style. > Use short color names. > > Signed-off-by: Sergey Ponomarev <stok...@gmail.com> > --- > networking/httpd_indexcgi.c | 19 ++++++++++++------- > 1 file changed, 12 insertions(+), 7 deletions(-) > > diff --git a/networking/httpd_indexcgi.c b/networking/httpd_indexcgi.c > index a94d4983e..b652ba9ff 100644 > --- a/networking/httpd_indexcgi.c > +++ b/networking/httpd_indexcgi.c > @@ -55,7 +55,6 @@ httpd_indexcgi.c -o index.cgi > "<style>" \ > "table {" \ > "width:100%;" \ > - "background-color:#fff5ee;" \ > "border-width:1px;" /* 1px 1px 1px 1px; */ \ > "border-spacing:2px;" \ > "border-style:solid;" /* solid solid solid solid; */ \ > @@ -76,9 +75,9 @@ httpd_indexcgi.c -o index.cgi > "border-color:black;" /* black black black black; */ \ > "white-space:nowrap" \ > "}" \ > -"tr.hdr { background-color:#eee5de }" \ > -"tr:nth-child(odd) { background-color:#ffffff }" \ > -"tr.foot { background-color:#eee5de }" \ > +"thead, tfoot { background-color:olive }" \ > +"tbody tr:nth-child(even) { background-color:beige }" \ > +"tbody tr:hover { background-color:ivory }" \
These colors look not that good to me. Can you separate change of colors and change in logic into two patches? _______________________________________________ busybox mailing list busybox@busybox.net http://lists.busybox.net/mailman/listinfo/busybox