kdereview: adjustable clock

[Emdek]

On 26-08-2010 at 16:38:23 Sebastian Kügler <sebas at kde.org> wrote:
> On Wednesday, August 25, 2010 01:37:45 pm Emdek wrote:
>> > Or just switch off reflections when in non-Planar mode?
>>
>> Switching off parts of code is bad idea, we should not force user to not
>> use something if we don't know how he want to use it.
>
> Plasma's FormFactor is actually designed to give the applet hints how it
> should behave. If it's in a horizontal panel (i.e. the available vertical
> space is constrained), optimizing it for vertical usage of space is the  
> right
> thing to do. If there is the space (which FormFactor::Planar means), you  
> can
> drop eyecandy around it to make it more readable (more space for actual
> content).
>
> You'll hit a hard limit at some point anyway, namely when the space you  
> have
> available for text is smaller than the height of
> KGlobalSettings::SmallestReadableFont.

Well, this is not relevant in this case, all rendering is done by WebKit,  
user only specifies HTML to feed it and only interactions done by applet  
are setting default font color, check if background should be displayed  
(relevant for desktop) and calculation of scale factor to make it fit.
Trying to be smarter than user can cause only problems, if user mess up  
format then he simply can reset it to one of defaults ones and try to  
modify them again if he wants. :-)


-- 
Best regards,
Michal