kdereview: adjustable clock

[Marco Martin]

On Wednesday 25 August 2010, Emdek wrote:
> On 25-08-2010 at 13:30:49 Sebastian Kügler <sebas at kde.org> wrote:
> > On Wednesday 25 August 2010 13:16:17 Emdek wrote:
> >> On 24-08-2010 at 23:02:15 Aaron J. Seigo <aseigo at kde.org> wrote:
> >> > On Tuesday, August 24, 2010, Emdek wrote:
> >> >> What specifically doesn't work properly?
> >> > 
> >> > if i use the pre-built mode with the reflections in a panel of ~27px
> >> > high, it
> >> > doesn't fit.
> >> 
> >> Well, that format was added mostly to use on desktop (reflections on
> >> panels won't look nice anyway), but I can modify it to fit when I'll
> >> have
> >> some free time.
> > 
> > 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.
> Besides there is already version of this format without reflection. ;-)
> To fix reflection it should be enough to set fixed height or something
> similar, pure CSS and HTML (well, with some WebKit's additions).
> Fixed height is OK to use because contents are always scaled to fit.

or enable/disable reflections on resize constraint events based on the size,
but regardless where the plasmoid is designed to belong, to be distributed in 
the KDE release it must work reasonably well on all containments present in 
the KDE release.

/me thinkssome policies about that should be written down somewhere...

Cheers,
Marco Martin