<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="https://git.reviewboard.kde.org/r/116625/">https://git.reviewboard.kde.org/r/116625/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On March 6th, 2014, 2:06 a.m. UTC, <b>Hrvoje Senjan</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="https://git.reviewboard.kde.org/r/116625/diff/1/?file=252390#file252390line76" style="color: black; font-weight: bold; text-decoration: underline;">startkde.cmake</a>
<span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">76</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
</tr>
</tbody>
</table>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Why not just change the defaults in frameworkintegration instead of writing directly into users settings?</pre>
</blockquote>
<p>On March 6th, 2014, 6:21 a.m. UTC, <b>Martin Gräßlin</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;">I also don't like that this seems to be at least parsed with every startup. If it needs to go through our startup and cannot be done in frameworkintegration I think it should be a kconf update script.</pre>
</blockquote>
<p>On March 6th, 2014, 10:28 a.m. UTC, <b>Sebastian Kügler</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;">You mean the parsing of kdeglobals on startup of apps?
That would mean frameworkintegration depends on oxygen-font. I'm not sure that's an acceptable dependency, and I'd like to know for sure that that approach is acceptable before implementing it. CC:ing Alex Fiestas as the maintainer of that.
Alex would you be OK with a runtime dependency on Oxygen font in frameworkintegration?
</pre>
</blockquote>
</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">So we need at least some bits of this, anyway. We can only set the font face and size, but not the hinting method. We'll also need to fiddle with the fontpathes in startkde, that can't be done (sensibly) in frameworksintegration.
I've now moved the face and size setting to frameworksintegration and equipped the oxygen-fonts repo with the necessary cmake foo. I'll update this patch for the necessary work shortly.</pre>
<br />
<p>- Sebastian</p>
<br />
<p>On March 19th, 2014, 1:16 a.m. UTC, Sebastian Kügler wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Plasma and Àlex Fiestas.</div>
<div>By Sebastian Kügler.</div>
<p style="color: grey;"><i>Updated March 19, 2014, 1:16 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kde-workspace
</div>
<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;">Make Oxygen the default font
This has three portions: the fonts itself, installation of them including
fontconfig bits, and setting font defaults in kdeglobals.
Oxygen Font 0.4 Imported from oxygen-font repository.
The startkde portion contains the bits to write out a kdeglobals default
file if it doesn't exist with font settings applied. Usually, with install
prefix set to /usr, the installed oxygen font is found automatically by
fontconfig. If we're installed to a different prefix, we need to point
fontconfig at the font. We do that by linking it from either
XDG_DATA_HOME or ~/.fonts/ and updating fontconfig with it. The latter is
irrelevant for systems that install into /usr.
</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;">Logged into session with clean config. Font settings are applied correctly throughout.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>CMakeLists.txt <span style="color: grey">(0135bb1f7475862451775928adf5dc20167424e0)</span></li>
<li>fonts/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>fonts/oxygen/COPYING-GPL+FE.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>fonts/oxygen/GPL.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>fonts/oxygen/Oxygen-Sans-Bold.ttf <span style="color: grey">(PRE-CREATION)</span></li>
<li>fonts/oxygen/Oxygen-Sans.ttf <span style="color: grey">(PRE-CREATION)</span></li>
<li>fonts/oxygen/OxygenMono-Regular.ttf <span style="color: grey">(PRE-CREATION)</span></li>
<li>fonts/oxygen/README <span style="color: grey">(PRE-CREATION)</span></li>
<li>startkde.cmake <span style="color: grey">(53d1bc6a2098f634c5f386f95e1a1c504b554303)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/116625/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>