<br><br><div class="gmail_quote">On Mon, Nov 7, 2011 at 6:04 PM, Marc Delisle <span dir="ltr"><<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Madhura Jayaratne a écrit :<br>
<div class="im">><br>
><br>
> On Mon, Nov 7, 2011 at 4:09 PM, Marc Delisle <<a href="mailto:marc@infomarc.info">marc@infomarc.info</a><br>
</div><div><div></div><div class="h5">> <mailto:<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>>> wrote:<br>
><br>
> Le 2011-11-06 07:51, Rouslan Placella a écrit :<br>
> > On Sun, 2011-11-06 at 06:54 -0500, Marc Delisle wrote:<br>
> >> Hi,<br>
> >><br>
> >> I would appreciate help about one display issue. In QA_3_4, browsing<br>
> >> sakila.actor with Firefox 7.0.1 (tested on Vista) shows vertical<br>
> solid<br>
> >> lines at the right of each data column. This is the expected<br>
> behavior,<br>
> >> produced by<br>
> >><br>
> >> #table_results td.data {<br>
> >> border-right: 1px solid #444444;<br>
> >> }<br>
> >><br>
> >> However in master, the lines appear then disappear. It happens<br>
> somewhere<br>
> >> in makegrid.js.<br>
> >><br>
> ><br>
> > Hi Marc,<br>
> ><br>
> > the issue is that each table column is overlayed with a div<br>
> element and<br>
> > they have conflicting border-right CSS declarations. Fix below,<br>
> adjust<br>
> > it as you see fit, if needed.<br>
> ><br>
> ><br>
> <a href="http://repo.or.cz/w/phpmyadmin/roccivic.git/commitdiff/4d1e6f302f04543ce1fea0524d3d8179ee894d21" target="_blank">http://repo.or.cz/w/phpmyadmin/roccivic.git/commitdiff/4d1e6f302f04543ce1fea0524d3d8179ee894d21</a><br>
> ><br>
> > Or pull from branch "colborder" of<br>
> > git://<a href="http://repo.or.cz/phpmyadmin/roccivic.git" target="_blank">repo.or.cz/phpmyadmin/roccivic.git</a><br>
</div></div>> <<a href="http://repo.or.cz/phpmyadmin/roccivic.git" target="_blank">http://repo.or.cz/phpmyadmin/roccivic.git</a>><br>
<div class="im">><br>
> I have pushed to master, thanks. I remain open to suggestions on<br>
> improving this visual column separation.<br>
><br>
><br>
> Hi,<br>
><br>
> I did not have any problem with the old white borders since they were<br>
> visible over the alternating ash colored rows. And the alignment also<br>
> contributed to the visual column separation.<br>
> However if you guys think we should go for a dark border, I prefer<br>
> #BBBBBB much over black.<br>
<br>
</div>Madhura,<br>
some ACL monitors did not show the previous alternating color for rows<br>
and the white border.<br>
<div><div></div><div class="h5"><br></div></div></blockquote></div><div>In that case I guess it's OK to either make the row color more contrast (keeping the white borders) or to use the #BBBBBB colored borders. </div>
<div>But if we are going for the second options don't we need horizontal borders as well? Unfortunately I don't have an ACL to see how it really looks. </div><div><br></div>-- <br>Thanks and Regards,<div><br></div>
<div>Madhura Jayaratne<br><div><br></div></div><br>