senjuhashirama / pugixml

Automatically exported from code.google.com/p/pugixml
0 stars 0 forks source link

The CSS file supplied in the docs could do with improvement... update supplied #199

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
View the supplied HTML documentation (offline)

What is the expected output? What do you see instead?
Font size is too small for comfort (for me).

The line spacing (line-height:) doesn't have a unit, and at least some (Chrome) 
browsers therefore ignore it. It appears the unit is expected to be "em". 

There are several classes that the css doesn't define, which means the inline 
elements - filenames, comments, etc - are at the mercy of default definitions.

Which version of pugixml are you using? On what operating system/compiler?
1.2 tarball.

Please provide any additional information below.
Provided reworked css file which 

Original issue reported on code.google.com by rivimeyc...@gmail.com on 3 Mar 2013 at 1:47

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by arseny.k...@gmail.com on 6 Mar 2013 at 4:04

GoogleCodeExporter commented 9 years ago
The attached file looks worse for me (probably because of a different font 
family and slightly larger font?..).

pugixml.css was a direct copy of this file: 
http://www.boost.org/doc/libs/1_54_0/doc/src/boostbook.css; the file in 
question changed a bit since then - the font sizes in the file are now relative.

I'm okay with updating pugixml.css with a newer copy of boostbook.css. Does 
boost documentation look good to you?

Original comment by arseny.k...@gmail.com on 10 Jul 2013 at 4:56

GoogleCodeExporter commented 9 years ago

Original comment by arseny.k...@gmail.com on 20 Jul 2013 at 5:59

GoogleCodeExporter commented 9 years ago
After taking a closer look, documentation looks the same with the updated 
boostbook.css - the font sizes for most elements are exactly the same. I'm 
closing this issue, since an alternative css does not seem to be readily 
available.

Original comment by arseny.k...@gmail.com on 30 Jul 2013 at 12:58