WWW-Changes archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: htdocs



On 2019-W36-4 09:26 +0000, Martin Husemann wrote:
On Wed, Sep 04, 2019 at 11:35:22PM +0000, Constantine A. Murenin wrote:
> Module Name:       htdocs
> Committed By:      cnst
> Date:              Wed Sep  4 23:35:21 UTC 2019
> > Modified Files:
>    htdocs: global.css
> > Log Message:
> global.css: add border to th, td elements; improve table.developerslist

This makes the releng.NetBSD.org/cgi-bin/builds.cgi table look very
horrible.

I think it just makes it look different.

Is the objection against any borders for any tables in the first place, or specifically against the border around each td element (e.g., against two 1px lines separated by a bit of whitespace around each cell)?

I think tables without borders just look wrong, although in the specific example you mention the table is not generated correctly, because the "CVS tag" doesn't actually span all the rows that it refers to, resulting in many empty cells, so, it's certainly not the best application of the border. Do you know where the code for this script is, perhaps we could fix the script to generate the table correctly?

<maya> pointed out this also breaks mail-index, as it's still using the table element to mimick column-based non-table data.

How about trying the borders on all pages that have been converted to the new design, so it doesn't apply to mail-index.NetBSD.org? (With or without a border-collapse that avoids having duplicate borders between each cell?)

The following code would apply to content on wiki.NetBSD.org, www.NetBSD.org and http://releng.netbsd.org/cgi-bin/builds.cgi, but not on mail-index.NetBSD.org. It results in a 3px border between header cells, and a single 1px border total between all other cells.

#content th {
        border: 3px solid #eee;
}
#content td {
        border: 1px solid #ddd;
}
#content table {
        border-collapse: collapse;
}

C.


Home | Main Index | Thread Index | Old Index