Re: [PATCH 6/7] docs: css: Modify appearance of the kbase directory page

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

 



On a Monday in 2020, Peter Krempa wrote:
On Mon, Aug 03, 2020 at 14:30:30 +0200, Ján Tomko wrote:
On a Monday in 2020, Peter Krempa wrote:
> Remove the gray background and switch to underlined links.
>

Here, I prefer the old look. The difference with the ToC might be
that the ToC entries are indented.

Both links and background? In case you prefer the old links I might want
to get rid of the background at least.


Sorry, I only compared the overall look.

After some experimenting, what concerns me most is the spacing:

  In the old version the gray panel aligns with the 'Knowledge base'
heading.

  In the white one, the K aligns with 'Debug logs' and it's too close
to the heading.

So I miss mostly:
  padding: 0.5 em;
that came from the panel.

1. I'm okay with dropping the gray background, since the kbase page is
different. (The bikeshedding question is: will we eventually fill it
enough to warrant a return of the gray panels and put three of them
on a computer screen?)

2. The 0.5 em padding should stay (I guess keeping another panel class
for this makes sense then.

3. The links look slightly better and more consistent with the rest of
the website with text-decoration: none; but I don't have a strong
preference here - we do have underlined links on white background
elsewhere.

Unrelated to this patch:

4. We have a 0.5 em padding for both `div.panel dl` and `div.panel p`,
which add up to 1 em for the paragraphs inside `dd`. Since what
we're adding there should be a one-sentence description of the document,
can we somehow convince the rst tools not to put a paragraph there?
Or just adjust the padding back for `div.panel p dd`? HTML, I'll be
missing you.

Jano

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux