From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ferruh Yigit <ferruh.yigit@intel.com>
Subject: Re: [PATCH 0/3] cross focus in NIC overview HTML table
Date: Thu, 23 Feb 2017 11:00:40 +0000
Message-ID: <4ab13054-d072-f0da-786e-481f132ab76c@intel.com>
References: <1487625998-8040-1-git-send-email-thomas.monjalon@6wind.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: 7bit
To: Thomas Monjalon <thomas.monjalon@6wind.com>, dev@dpdk.org
Return-path: <dev-bounces@dpdk.org>
Received: from mga09.intel.com (mga09.intel.com [134.134.136.24])
 by dpdk.org (Postfix) with ESMTP id 527E75598
 for <dev@dpdk.org>; Thu, 23 Feb 2017 12:00:43 +0100 (CET)
In-Reply-To: <1487625998-8040-1-git-send-email-thomas.monjalon@6wind.com>
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

On 2/20/2017 9:26 PM, Thomas Monjalon wrote:
> In order to make the NIC overview easier to read
> (http://dpdk.org/doc/guides/nics/overview.html)
> the row and column pointed by the mouse are highlighted.
> 
> Thomas Monjalon (3):
>   doc: set alignments in NIC overview table
>   doc: highlight pointed row in NIC overview table
>   doc: highlight pointed column in NIC overview table

Series Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>

<...>

> A next step could be to highlight every implemented features when
> pointing a driver, or every drivers implenting a pointed feature.
> The latter could be done by this patch:
> 
>        table#id1 th:not(:first-child):hover::after,
> -      table#id1 td:not(:first-child):hover::after {
> +      table#id1 td:not(:first-child):hover::after,
> +      table#id1 td:first-child:hover ~ td:not(:empty)::after {
>           content: '';
>           height: 6000px;
>           top: -3000px;
> 
> Unfortunately, I cannot make it work, probably because the cells are not
> really empty. A javascript function could help to trim the cells.
> Any help to implement this next step would be welcome.

Overall I agree that some readability improvements would be good for NIC
overview table.