[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: Review Request 113419: Remove the upper-half white gradient from KSplash Minimalistic theme
From: Thomas_Lübking <thomas.luebking () gmail ! com>
Date: 2013-10-24 17:06:26
Message-ID: 20131024170626.13629.11013 () vidsolbach ! de
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
> On Oct. 24, 2013, 3:35 p.m., Ivan Čukić wrote:
> > Actually, this is something that I wanted to do ever since I saw it on a different screen \
> > to the one I had when I made it.
> > If you want to try it with a radial gradient, mind that around the logo, the background \
> > needs to be pitch-black. Because the masking for the gear is a simple black overlay.
>
> Ivan Čukić wrote:
> p.s. And it *needs* some background noise so that the gradients look less artificial.
That's not really gonna work.
You need to apply an actual distorsion filter on the gradient (ie. randomly move around quite \
some pixels) to avoid banding if you've a gradient across a thousand pixels that spans from \
#000000 to #111111
So you'd rather bring a prefabbed image and at best don't scale it, but preferably scale it UP \
(more pixels), not down (more banding) Then plug the KDE logo in front of it.
- Thomas
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113419/#review42292
-----------------------------------------------------------
On Oct. 24, 2013, 3:15 p.m., Martin Klapetek wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/113419/
> -----------------------------------------------------------
>
> (Updated Oct. 24, 2013, 3:15 p.m.)
>
>
> Review request for kde-workspace and Plasma.
>
>
> Repository: kde-workspace
>
>
> Description
> -------
>
> NOTE: This is intended for the PW2, not current 4.x series.
>
> Removes the white-black gradient spreading through the upper-half of the screen. IMHO it \
> looks more elegant when it's just plain black.
>
> Diffs
> -----
>
> ksplash/ksplashqml/themes/Minimalistic/main.qml e4fb8b8
>
> Diff: http://git.reviewboard.kde.org/r/113419/diff/
>
>
> Testing
> -------
>
>
> File Attachments
> ----------------
>
> Before
> http://git.reviewboard.kde.org/media/uploaded/files/2013/10/24/32c5b408-0d3e-4639-b2ae-18f1a8dbd699__ksp_old.png
> After
> http://git.reviewboard.kde.org/media/uploaded/files/2013/10/24/d494e4b4-6e13-4ee8-934c-6ff7ac21bc33__ksp_new.png
>
>
> Thanks,
>
> Martin Klapetek
>
>
[Attachment #5 (text/html)]
<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/113419/">http://git.reviewboard.kde.org/r/113419/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On October 24th, 2013, 3:35 p.m. UTC, <b>Ivan Čukić</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;">Actually, this is something that I wanted to \
do ever since I saw it on a different screen to the one I had when I made it.
If you want to try it with a radial gradient, mind that around the logo, the background needs \
to be pitch-black. Because the masking for the gear is a simple black overlay.</pre> \
</blockquote>
<p>On October 24th, 2013, 3:56 p.m. UTC, <b>Ivan Čukić</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;">p.s. And it *needs* some background noise so \
that the gradients look less artificial.</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;">That's not really gonna work. You need to \
apply an actual distorsion filter on the gradient (ie. randomly move around quite some pixels) \
to avoid banding if you've a gradient across a thousand pixels that spans from #000000 to \
#111111
So you'd rather bring a prefabbed image and at best don't scale it, but preferably \
scale it UP (more pixels), not down (more banding) Then plug the KDE logo in front of it.</pre>
<br />
<p>- Thomas</p>
<br />
<p>On October 24th, 2013, 3:15 p.m. UTC, Martin Klapetek wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: \
url('http://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 kde-workspace and Plasma.</div>
<div>By Martin Klapetek.</div>
<p style="color: grey;"><i>Updated Oct. 24, 2013, 3:15 p.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;">NOTE: This is \
intended for the PW2, not current 4.x series.
Removes the white-black gradient spreading through the upper-half of the screen. IMHO it looks \
more elegant when it's just plain black.</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>ksplash/ksplashqml/themes/Minimalistic/main.qml <span style="color: \
grey">(e4fb8b8)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/113419/diff/" style="margin-left: 3em;">View \
Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>
<ul>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/10/24/32c5b408-0d3e-4639-b2ae-18f1a8dbd699__ksp_old.png">Before</a></li>
<li><a href="http://git.reviewboard.kde.org/media/uploaded/files/2013/10/24/d494e4b4-6e13-4ee8-934c-6ff7ac21bc33__ksp_new.png">After</a></li>
</ul>
</td>
</tr>
</table>
</div>
</body>
</html>
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic