Re: [libvirt] [PATCH] docs: Update style for <dt> elements

On Thu, 2016-04-21 at 08:04 -0400, John Ferlan wrote: > Add bolding for <dt> elements to make them "stick out" on the page rather > that just a stream of text where the <dt> elements only differ by slightly > different font style since most uses encase the text within using <code> > > Signed-off-by: John Ferlan <jferlan(a)redhat.com&gt; > --- > > I had suggested this during a review of Andrea's changes late last month: > > http://www.redhat.com/archives/libvir-list/2016-March/msg01501.html > > but figure I'll make the more 'formal' patch request... Yeah, sorry about that. I just forgot :(

> docs/generic.css | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/docs/generic.css b/docs/generic.css > index 208e31e..b86bb88 100644 > --- a/docs/generic.css > +++ b/docs/generic.css > @@ -25,6 +25,7 @@ ul, ol { > dt { > margin-left: 1em; > margin-right: 2em; > + font-weight: bold; > } > > dl dd { This styling is appropriate when the content of <dt> is a <code> element, because that makes the font smaller. This is the case for most of the <dt>s in our documentation, but not *all* of them. I've just posted a patch[1] that fixes a bunch of <dt>s that were missing the inner <code> element, but that still leaves out a few that are just not supposed to have it - look no further than contact.html for an example. So my proposal is, apply my patch first, and then add dt code { font-weight: bold; } to the stylesheet instead. Sounds good?

[1] https://www.redhat.com/archives/libvir-list/2016-April/msg01476.html -- Andrea Bolognani / Red Hat / Virtualization Team