<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0//EN" "http://www.w3.org/TR/REC-html40/strict.dtd"><html><head><meta name="qrichtext" content="1" /><style type="text/css">p, li { white-space: pre-wrap; }</style></head><body style=" font-family:'Consolas'; font-size:11pt; font-weight:400; font-style:normal;">On Thursday 19 June 2008, Urs Wolfer wrote:<br>> On Thursday 19 June 2008 01:25:27 Michael Pyne wrote:<br>> > I've made a patch that fixes it and makes it look better (at least, IMO).<br>> > Both the headers and footers are corrected and somewhat more employed.<br>> > I've tried to make the appearance more like www.kde.org as well.<br>><br>> Good work. I just would like to have the same style as all the others apps<br>> with "HTML"-pages does have (e.g. all the start pages, see my other mail in<br>> this thread). It would be a pitty to just have a different style here.<br>> Either update all (please talk to the oxygen artists first), or try to<br>> follow the current look.<br><p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"></p>I think the objection to the Konq page is because of the off-white thin line which you can see coming out to the right from the logo, extending under the "Konqueror" text. It is the same color as the background of the descriptive text but is not the same color as the background, which looks distracting.<br><p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"></p>In addition I really don't think the Konqueror/KMail example would apply to the man page output, as the text would have to be artifically confined to the black box which would make the output much longer than it otherwise has to be.<br><p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"></p>I chose the www.kde.org site because it is the current example of what KDE uses to distribute lots of information over HTML (TechBase, for instance, has much the same style), as opposed to using the same stuff used for the Konqueror start page, as that seems intended to be more of a pretty "Hello!" widget than a general purpose HTML presentation style sheet (indeed, the components for that are in the "/about" directory of kdeui, implying that they are a fancy About box).<br><p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"></p>> > I had to add a style sheet which I simply dumped into a header file, it<br>> > can't go straight into a installed css as it needs some munging to insert<br>> > the correct path to the doc/HTML directory.<br>><br>> You could take the stylesheet which is installed from kdeui (see for<br>> example KMail or Konqueror) instead of copying this to a second place.<br><p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"></p>The problem comes because the image files required are pulled in from the CSS. This could be rectified by simply appending the correct CSS style rules in the generated HTML so that straight .css could be used, so I will rework the patch to do that. But I still wouldn't be able to use the stylesheets from kdeui/about.<br><p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"></p>Thanks for the suggestions though. If no one else has any objections in the next few hours or so I'll probably go ahead and commit, we can integrate the various styles later if we decide to go that route.<br><p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"></p>Regards,<br> - Michael Pyne</p></body></html>