[Phpmyadmin-devel] makegrid.js help

Hi, I would appreciate help about one display issue. In QA_3_4, browsing sakila.actor with Firefox 7.0.1 (tested on Vista) shows vertical solid lines at the right of each data column. This is the expected behavior, produced by #table_results td.data { border-right: 1px solid #444444; } However in master, the lines appear then disappear. It happens somewhere in makegrid.js. -- Marc Delisle http://infomarc.info

On Sun, 2011-11-06 at 06:54 -0500, Marc Delisle wrote:
Hi,
I would appreciate help about one display issue. In QA_3_4, browsing sakila.actor with Firefox 7.0.1 (tested on Vista) shows vertical solid lines at the right of each data column. This is the expected behavior, produced by
#table_results td.data { border-right: 1px solid #444444; }
However in master, the lines appear then disappear. It happens somewhere in makegrid.js.
Hi Marc, the issue is that each table column is overlayed with a div element and they have conflicting border-right CSS declarations. Fix below, adjust it as you see fit, if needed. http://repo.or.cz/w/phpmyadmin/roccivic.git/commitdiff/4d1e6f302f04543ce1fea... Or pull from branch "colborder" of git://repo.or.cz/phpmyadmin/roccivic.git Rouslan

Le 2011-11-06 07:51, Rouslan Placella a écrit :
On Sun, 2011-11-06 at 06:54 -0500, Marc Delisle wrote:
Hi,
I would appreciate help about one display issue. In QA_3_4, browsing sakila.actor with Firefox 7.0.1 (tested on Vista) shows vertical solid lines at the right of each data column. This is the expected behavior, produced by
#table_results td.data { border-right: 1px solid #444444; }
However in master, the lines appear then disappear. It happens somewhere in makegrid.js.
Hi Marc,
the issue is that each table column is overlayed with a div element and they have conflicting border-right CSS declarations. Fix below, adjust it as you see fit, if needed.
http://repo.or.cz/w/phpmyadmin/roccivic.git/commitdiff/4d1e6f302f04543ce1fea...
Or pull from branch "colborder" of git://repo.or.cz/phpmyadmin/roccivic.git
I have pushed to master, thanks. I remain open to suggestions on improving this visual column separation. In the 3.3 branch (which did not have pmahomme), there was a more subtle vertical separation between column but it was there. -- Marc Delisle http://infomarc.info

On Mon, Nov 7, 2011 at 4:09 PM, Marc Delisle <marc@infomarc.info> wrote:
Le 2011-11-06 07:51, Rouslan Placella a écrit :
On Sun, 2011-11-06 at 06:54 -0500, Marc Delisle wrote:
Hi,
I would appreciate help about one display issue. In QA_3_4, browsing sakila.actor with Firefox 7.0.1 (tested on Vista) shows vertical solid lines at the right of each data column. This is the expected behavior, produced by
#table_results td.data { border-right: 1px solid #444444; }
However in master, the lines appear then disappear. It happens somewhere in makegrid.js.
Hi Marc,
the issue is that each table column is overlayed with a div element and they have conflicting border-right CSS declarations. Fix below, adjust it as you see fit, if needed.
http://repo.or.cz/w/phpmyadmin/roccivic.git/commitdiff/4d1e6f302f04543ce1fea...
Or pull from branch "colborder" of git://repo.or.cz/phpmyadmin/roccivic.git
I have pushed to master, thanks. I remain open to suggestions on improving this visual column separation.
Hi, I did not have any problem with the old white borders since they were visible over the alternating ash colored rows. And the alignment also contributed to the visual column separation. However if you guys think we should go for a dark border, I prefer #BBBBBB much over black. -- Thanks and Regards, Madhura Jayaratne

Madhura Jayaratne a écrit :
On Mon, Nov 7, 2011 at 4:09 PM, Marc Delisle <marc@infomarc.info <mailto:marc@infomarc.info>> wrote:
Le 2011-11-06 07:51, Rouslan Placella a écrit : > On Sun, 2011-11-06 at 06:54 -0500, Marc Delisle wrote: >> Hi, >> >> I would appreciate help about one display issue. In QA_3_4, browsing >> sakila.actor with Firefox 7.0.1 (tested on Vista) shows vertical solid >> lines at the right of each data column. This is the expected behavior, >> produced by >> >> #table_results td.data { >> border-right: 1px solid #444444; >> } >> >> However in master, the lines appear then disappear. It happens somewhere >> in makegrid.js. >> > > Hi Marc, > > the issue is that each table column is overlayed with a div element and > they have conflicting border-right CSS declarations. Fix below, adjust > it as you see fit, if needed. > > http://repo.or.cz/w/phpmyadmin/roccivic.git/commitdiff/4d1e6f302f04543ce1fea... > > Or pull from branch "colborder" of > git://repo.or.cz/phpmyadmin/roccivic.git <http://repo.or.cz/phpmyadmin/roccivic.git>
I have pushed to master, thanks. I remain open to suggestions on improving this visual column separation.
Hi,
I did not have any problem with the old white borders since they were visible over the alternating ash colored rows. And the alignment also contributed to the visual column separation. However if you guys think we should go for a dark border, I prefer #BBBBBB much over black.
Madhura, some ACL monitors did not show the previous alternating color for rows and the white border. -- Marc Delisle http://infomarc.info

On Mon, Nov 7, 2011 at 6:04 PM, Marc Delisle <marc@infomarc.info> wrote:
Madhura Jayaratne a écrit :
On Mon, Nov 7, 2011 at 4:09 PM, Marc Delisle <marc@infomarc.info <mailto:marc@infomarc.info>> wrote:
Le 2011-11-06 07:51, Rouslan Placella a écrit : > On Sun, 2011-11-06 at 06:54 -0500, Marc Delisle wrote: >> Hi, >> >> I would appreciate help about one display issue. In QA_3_4,
browsing
>> sakila.actor with Firefox 7.0.1 (tested on Vista) shows vertical solid >> lines at the right of each data column. This is the expected behavior, >> produced by >> >> #table_results td.data { >> border-right: 1px solid #444444; >> } >> >> However in master, the lines appear then disappear. It happens somewhere >> in makegrid.js. >> > > Hi Marc, > > the issue is that each table column is overlayed with a div element and > they have conflicting border-right CSS declarations. Fix below, adjust > it as you see fit, if needed. > >
http://repo.or.cz/w/phpmyadmin/roccivic.git/commitdiff/4d1e6f302f04543ce1fea...
> > Or pull from branch "colborder" of > git://repo.or.cz/phpmyadmin/roccivic.git <http://repo.or.cz/phpmyadmin/roccivic.git>
I have pushed to master, thanks. I remain open to suggestions on improving this visual column separation.
Hi,
I did not have any problem with the old white borders since they were visible over the alternating ash colored rows. And the alignment also contributed to the visual column separation. However if you guys think we should go for a dark border, I prefer #BBBBBB much over black.
Madhura, some ACL monitors did not show the previous alternating color for rows and the white border.
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. 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.
-- Thanks and Regards, Madhura Jayaratne

Madhura Jayaratne a écrit :
On Mon, Nov 7, 2011 at 6:04 PM, Marc Delisle <marc@infomarc.info <mailto:marc@infomarc.info>> wrote:
Madhura Jayaratne a écrit : > > > On Mon, Nov 7, 2011 at 4:09 PM, Marc Delisle <marc@infomarc.info <mailto:marc@infomarc.info> > <mailto:marc@infomarc.info <mailto:marc@infomarc.info>>> wrote: > > Le 2011-11-06 07:51, Rouslan Placella a écrit : > > On Sun, 2011-11-06 at 06:54 -0500, Marc Delisle wrote: > >> Hi, > >> > >> I would appreciate help about one display issue. In QA_3_4, browsing > >> sakila.actor with Firefox 7.0.1 (tested on Vista) shows vertical > solid > >> lines at the right of each data column. This is the expected > behavior, > >> produced by > >> > >> #table_results td.data { > >> border-right: 1px solid #444444; > >> } > >> > >> However in master, the lines appear then disappear. It happens > somewhere > >> in makegrid.js. > >> > > > > Hi Marc, > > > > the issue is that each table column is overlayed with a div > element and > > they have conflicting border-right CSS declarations. Fix below, > adjust > > it as you see fit, if needed. > > > > > http://repo.or.cz/w/phpmyadmin/roccivic.git/commitdiff/4d1e6f302f04543ce1fea... > > > > Or pull from branch "colborder" of > > git://repo.or.cz/phpmyadmin/roccivic.git <http://repo.or.cz/phpmyadmin/roccivic.git> > <http://repo.or.cz/phpmyadmin/roccivic.git> > > I have pushed to master, thanks. I remain open to suggestions on > improving this visual column separation. > > > Hi, > > I did not have any problem with the old white borders since they were > visible over the alternating ash colored rows. And the alignment also > contributed to the visual column separation. > However if you guys think we should go for a dark border, I prefer > #BBBBBB much over black.
Madhura, some ACL monitors did not show the previous alternating color for rows and the white border.
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.
I have merged the change to #BBBBBB for now.
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.
On ACL, there is no problem with horizontal rows, due to the new alternating color in pmahomme. -- Marc Delisle http://infomarc.info
participants (3)
-
Madhura Jayaratne
-
Marc Delisle
-
Rouslan Placella