<div dir="ltr">I think we should remove the kde icon with the plasma one.<br></div><div class="gmail_extra"><br><div class="gmail_quote">2016-03-11 10:59 GMT+01:00 Harald Sitter <span dir="ltr"><<a href="mailto:sitter@kde.org" target="_blank">sitter@kde.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thu, Mar 10, 2016 at 6:25 PM, Jonathan Riddell <<a href="mailto:jr@jriddell.org">jr@jriddell.org</a>> wrote:<br>
> On Thu, Mar 10, 2016 at 05:10:40PM +0000, Jonathan Riddell wrote:<br>
</span><span class="">>> Some queries:<br>
>>  - why does it have a .grub theme file? shouldn't that be in breeze-grub?<br>
<br>
</span>Actually I think we can drop it, it is some debian coloring tech that<br>
ultimately is useless if one uses our grub theme. Unless someone knows<br>
why we would want to keep it?<br>
<span class=""><br>
>>  - It doesn't do anything for ask-question<br>
<br>
</span>A bug! Oh my. Turns out it is not implemented at all. I also struggle<br>
to find an actual use of the command though. Also that will take<br>
additional vertical space we do not have -.-<br>
<span class=""><br>
>>  - It depends on plymouth-label but doesn't check for it at configure time (maybe it's just Debian splitting it out when other distros don't)<br>
<br>
</span>Plymouth-label is a runtime plugin. I take it you think cmake should<br>
check for it as RUNTIME dependency?<br>
<span class=""><br>
>>  - There's graphics for distros to have their own logo in place of the KDE one but no means to use that, are distros expected to just patch them?<br>
<br>
</span>Yep. See README. Ideally what I would like to have is a build-time<br>
asset compiler to both adjust the .script and merge images/ with a<br>
branded variant of it. For now patching should do though.<br>
<span class=""><br>
> And the text covers the progress spinner<br>
>  <a href="https://www.flickr.com/photos/jriddell/25650567456/" rel="noreferrer" target="_blank">https://www.flickr.com/photos/jriddell/25650567456/</a><br>
<br>
</span>This appears to be a spacing issue on 800x600, supposedly it gets<br>
worse for 640x480. I did fix the text spacing to just barely make it<br>
on 600p (with a bit of overlap) that's still going to phenomenally<br>
fail for 480p though. Options at this point are<br>
<br>
- don't use logo on low res<br>
- use lower res logo on low res (this actually might be best - on 480<br>
the logo would consume 1/4 of the height with the logo aligned at the<br>
vertical center that would leave the title, spinner and text with a<br>
glorious 180 pixels :O)<br>
- move the entire layout up when it runs out of space. while<br>
technically very appealing since plymouth doesn't have any layout tech<br>
besides placing things by x/y/z this actually would be the most<br>
invasive approach as we have to build adaptive layout rigging in<br>
plyscript.<br>
<br>
None of which seems terribly compelling, lower res logos certainly<br>
seem the most likely choice. They may increase the theme size a bit,<br>
but improve appearance.<br>
<br>
I quickly tried the logo resolution approach since that is fairly<br>
cheap to do code-wise:<br>
<a href="http://imgur.com/a/DEV1S" rel="noreferrer" target="_blank">http://imgur.com/a/DEV1S</a><br>
that's 480 with a 32x32 logo and 600 with a 64x64 logo. Along with the<br>
spacing fix I did 600 looks fairly decent this way, 480 remains a<br>
hopeless mess.<br>
<span class="HOEnZb"><font color="#888888"><br>
HS<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
Plasma-devel mailing list<br>
<a href="mailto:Plasma-devel@kde.org">Plasma-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/plasma-devel" rel="noreferrer" target="_blank">https://mail.kde.org/mailman/listinfo/plasma-devel</a><br>
</div></div></blockquote></div><br></div>