Review Request 113419: Remove the upper-half white gradient from KSplash Minimalistic theme

Martin Klapetek martin.klapetek at gmail.com
Thu Oct 24 15:14:02 UTC 2013



> On Oct. 24, 2013, 3:08 p.m., Mark Gaiser wrote:
> > -1
> > 
> > I have to add some explanation to that.
> > 
> > I'm all OK with plain and simplistic, but not TOO plain and simplistic (aka, the Windows 8 and apple route). We don't live in a monochrome age anymore so i think the theme should retain "some" subtle color variations. Having said that, the current plain horizontal top gradient might be a bit too simple (as in not very stylish) as well. Take a look at this for example: http://paswall.com/black-wallpaper-6462-hd.html Ignore the texture, but you can see the subtle lighting in there. That's probably what the plain theme needs.
> > 
> > This is just my opinion (and i happen to hate iOS7 + windows 8 for their fugly themes). I disagree with the change, but understand it if you would push it regardless. The general design theme these days seems to be heading back to 16 color monitors anyway.

Yeah, the radial gradient could be nice. Can QML do radial gradients?


- Martin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113419/#review42286
-----------------------------------------------------------


On Oct. 24, 2013, 2:32 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, 2:32 p.m.)
> 
> 
> Review request for kde-workspace and Plasma.
> 
> 
> Repository: kde-workspace
> 
> 
> Description
> -------
> 
> 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
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20131024/e3212901/attachment.html>


More information about the Plasma-devel mailing list