Opened 21 years ago

Closed 21 years ago

#812 closed defect (fixed)

Proposal to modify the book TOC in newxml

Reported by: archaic@… Owned by: lfs-book@…
Priority: lowest Milestone:
Component: Book Version: CVS
Severity: normal Keywords:
Cc:

Description

I've made a couple of changes to the default generated index.html as a POC. This cleans up the index a bit both for the eyes and to reduce redundant links. I only made the change in the HTML because I couldn't figure out how to do it in the xml. The proposal is at: http://www.linuxfromscratch.org/~archaic/index-proposed.html

Attachments (2)

index.html (30.3 KB ) - added by manuel@… 21 years ago.
First testcase
index.2.html (24.0 KB ) - added by manuel@… 21 years ago.
Second testcase

Download all attachments as: .zip

Change History (12)

comment:1 by archaic@…, 21 years ago

Forgot to mention that some judicious use of <h?> tags might make the sections stand out a little nicer as well.

comment:2 by manuel@…, 21 years ago

This need some disscusion in the list and a editorial decision before make the chages in the

templates. Also there are some issues in the used tags that should be fixed, noted be Jeroen in this post: http://archives.linuxfromscratch.org/mail-archives/lfs-dev/2004-April/044552.html As a side note, archiac, please change in the Summary of this bug "index" by "TOC", I can't do it.

comment:3 by archaic@…, 21 years ago

rep_platform: PCAll
Summary: Proposal to modify the book index in newxmlProposal to modify the book TOC in newxml

comment:4 by archaic@…, 21 years ago

Changed Summary title.

by manuel@…, 21 years ago

Attachment: index.html added

First testcase

comment:5 by jeroen@…, 21 years ago

Looks good, though I'm not sure why all links need to be <h4>. If you'd like to get all headers equal in the book it would make sense, but I think the use of <h?> should be decided based on the context of a page. In this context, the <h4>'s aren't real headers of anything. Thus, I recommend dropping the use of <h4> for the links.

by manuel@…, 21 years ago

Attachment: index.2.html added

Second testcase

comment:6 by jeroen@…, 21 years ago

Looks great, good work!

comment:7 by manuel@…, 21 years ago

Added both propossals to the full render of the book and sources that can be found here:

http://www.lfs-es.org/newxml

comment:8 by Matthew Burgess, 21 years ago

Manuel, is this ready to go? It looks great, but before I apply your patch I'd like you to confirm there's no outstanding issues. The only thing left that I can think of is removing "Installing" from our package installation pages. So "Installing Autoconf-2.59" simply becomes "Autoconf-2.59".

comment:9 by manuel@…, 21 years ago

Ready to go. This changes don't affect the XML code. It's controled by the "toc.list.type" param and

"toc.line" temple under the "TOC Stuff" section in stylesheets/lfs-chunked.xsl Complementary, removing the "Installing" word fron the title pages could improve the readability. Maybe we need also a CSS to fix the spaces betewn lines.

comment:10 by manuel@…, 21 years ago

Resolution: fixed
Status: newclosed

Fixed in the newxml module

Note: See TracTickets for help on using tickets.