<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/103426/">http://git.reviewboard.kde.org/r/103426/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On December 16th, 2011, 10:50 a.m., <b>Thomas Lübking</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">please use "20pt" or "1em" - even todays notebooks often have >=150dpi and you can expect this value to rise (macbook rumors...)</pre>
</blockquote>
<p>On December 16th, 2011, 2:50 p.m., <b>Thomas Lübking</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">errr... soryr for this probably stupid question, but why isn't the kde-defaults.css "fixed" then?
(just compared info:/ to man:/ - looks like blog.fefe.de where apparently everyone is supposed to use his own stylesheet ;-)
In other words and looking at the common header of man:/ and info:/ and for the fact that .chapter is handled there as well:
I guess the info:/ definition in kde-defaults.css is just not complete and i guess we should have a general central "corporate" design to align those looks, yesno?</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Because the content generated by each of the generators for those protocols has its own quirks and chaging the right padding inside a commonly shared CSS file like kde-default.css might inadvertantly break something else that uses it ?? For example, the man protocol most definitely does not this change and adding an additional 1em right padding would unnecessarily squish its content further right. Since the generated man pages are already formatted properly this change will most definitely have adverse effect on one of the many pages. The same applies to all the other places kde-default.css is used.
As far as I am concerned this issue is very specific to the kde-info2html generator and as such the fix applies to it alone. At least until someone has the time to throughly determine such change can safely be moved to a common location like kde-default.css.</pre>
<br />
<p>- Dawit</p>
<br />
<p>On December 16th, 2011, 2:15 p.m., Dawit Alemayehu wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for KDE Runtime.</div>
<div>By Dawit Alemayehu.</div>
<p style="color: grey;"><i>Updated Dec. 16, 2011, 2:15 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The patch adds a 20 px margin to info pages generated by kio_info to make them easier to read.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Tested by displaying info:/make/Goals.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=230959">230959</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>kioslave/info/kde-info2html <span style="color: grey">(03ace75)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/103426/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>