Bug 158762

Can't reproduce this with current svn trunk.

Created attachment 23784 [details]
My panel

Oh, if you have "Show date" actually *set* you can see about half of it. But it seems that if you remove the clock and then add it again that it forgets you had "Show date" set.

jonathan: that's because if you remove the clock and add another one it's a new clock with default settings :) you can have lots and lots of clocks with different settings this way.

Heh, figures. It would be cool if you could check an option that would make the applet settings default for all applets of it's kind. Maybe I should open up a wishlist report...

But if this is fixed in trunk, then it'd be neat if the fix was backported for 4.0.3.

So it simply doesn't fit, showing date and time. I assume it does work well if you switch off showing the date? The clock now tries to show the date (which is what you tell it to do) and respecting the smallest readable font size setting (from System Settings | Appearance | Fonts | Small). 

I can imagine the following:

 1) Silently remove the date when it doesn't fit anyway
 2) Ignore smallest readable font setting
 3) Take more horizontal space and show the date next to the time (in vertical panels, it would then just take more horizontal space.

I think I'd prefer 3), what do others think?

Maybe 3) and if textwidth>panelsize for vertical panel then 2) ? Probably, a "damn small text" is still better then a "half is not readable cause it's truncated" text? There may even be a 4) don't display e.g. year or seconds in the date if textwidth>panelsize and probably even a 5) split the string into multiple lines till it matches (so, e.g. "2008\n03\n07\n23\n59\n59") and a 6) combine some of the points into something ueber-clever :)

3)

And check how it works on vertical panels of various widths, please.

> ------- Heh, figures. It would be cool if you could check an option that
> would make the applet settings default for all applets of it's kind. Maybe
> I should open up a wishlist report...


sounds like a good idea to me :)

http://bugs.kde.org/show_bug.cgi?id=158909
Well there's the wishlist report if anyone was interested.

Oh, and that picture that I attached was *without* the date. So no, it doesn't render nicely without the date. (Or at least it doesn't in 4.0.2)

Also happen here - clock (only Time shown) does not fit in panel when panel is "Small" size.

Created attachment 23844 [details]
Wrong panel allignment

Same here. If I set the panel size to "tiny" the clock and all the panel icons
are  not alligned correclty. Some part of the panel is showed on top of the
screen.

Hi,

I have the same problem for the clock, and if I put the size of the panel to "tiny", the K menu is cut off too.
For the clock, we should have the choice of the alignment. something like that :

Hour:Min:(seconds)
Date

or 

Hour:Min:(seconds)  Date

Re: Comment #6

4) Make the time font smaller so that it fits.  If the time font then becomes smaller than the "smallest readable font", then generate an error message.

Created attachment 24117 [details]
Clock in vertical panel

FYI, here's what the clock looks like in a vertical panel in the KDE-based
Fedora 9 live image. Note that, in addition to being totally cut off, it's also
quite far from the bottom of the screen (the bottom of the image) and can't me
moved around.

Created attachment 24119 [details]
Clock in Vertical Panel 160 pixels

There seems to be a problem with the previous attachment.

For the clock to work in a vertical panel, the panel needs to be a reasonable
size.  So, I tried 160 pixels which I use in KDE-3.5.x.

This clearly shows the problems:

There is the AM/PM issue.  It isn't on the same line as the time numbers.

It also appears (in general) that always having the date the "Small" font is
not satisfactory.  The "Small" font being the minimum size for the date (and
time zone) should be one of the constraints used to determine the size.  Here
the font needs to be larger.

The size of the widget is wrong.  It appears to be vertical with a rather large
aspect ratio (height/width).  The clock takes up a lot of the panel space and
there is a large space between the time and the date.

There are clearly issues with Widgets.	It isn't possible for them to be the
same on the desktop, horizontal panel, or vertical panel.  They are going to
have to know which they are on and when on a panel apply size constraints which
are a function of vertical | horizontal.

If the "previous attachment" with problems is mine, then maybe I should clarify what it's meant to show. I only took a snapshot of the panel itself, not the whole screen; due to a different bug (apparently fixed now) the wallpaper was white at the time, so it does look a bit funny. But that's an honest image of what the lower-left corner of my screen looked like.

> There are clearly issues with Widgets.	It isn't possible for them to be the
> same on the desktop, horizontal panel, or vertical panel.  They are going
> to have to know which they are on and when on a panel apply size
> constraints which are a function of vertical | horizontal.


they already do just this.
the problem is that the digital clock is really, horrifically, stupendously 
*bad* at it. iirc there's all sorts of weird cruft from long ago when this 
stuff worked differently.

Created attachment 24124 [details]
vert. panel

can you explain me what's so wrong with vertically placed clock in vertically
placed panel?
e.g. why it has to be like this:
09:41
Sunday
30.03.2008

The vertical clock is half the screen high. That's the problem, in general.

the attachment I did takes about only 1/6 of my screen height

Oh, sorry, I didn't see the attachment. I though you were saying you weren't having any problems with the clock.

*** Bug 158827 has been marked as a duplicate of this bug. ***

*** Bug 157537 has been marked as a duplicate of this bug. ***

*** Bug 160286 has been marked as a duplicate of this bug. ***

Does the wrapping issue from comment 12 have its own bug report. If it doesn't, should it have its own separate report?

Created attachment 24273 [details]
Wrapped panel

The bug it's still present in 4.0.3.
I attached another screenshot to show that also kickoff is "wrapped" and it  is
show on top of the screen! Didn't notice this before...

Maybe my simil-wide-screen resolution (1280x800 on dell latitude d620) could be
the problem ?

Thanx!

-- 
Fabio

Same Problem like Fabio here. The Kickoff Menu is shown on top of the screen and the  bottom part of the Panel is also shown on the Top like in Fabios screenshot. 

I have the same resolution (1280x800) on an Acer TravelMate 2410 and an Intel 915 Mobile graphic card. Kubuntu 8.04 Beta with KDE 4.0.3

Thx

I have this problem as well with the Panel set to tiny.  KDE 4.0.3 and I'm using Nvidia MX400 1024x768.  

1.  Panel loads “Normal” size after KDE 4 install.

2.  Size is changed to “Tiny” size. (Clock it cut off)

3.  Size is changed back to “Normal” size. (The bottom half of the panel is white and the icons and clock are cut off)

Notice in the screen shots that KSnapshot always overlaps on the lower left of the task bar...  but that is another issue that needs addressed.

See my screenshots:

Created attachment 24525 [details]
Before Panel Resize

Created attachment 24526 [details]
After Panel Resize (Tiny)

Created attachment 24527 [details]
Panel Back To Normal Size

Created attachment 24528 [details]
Overlap of KSnapshot (Bonus)

Created attachment 24552 [details]
Missing panel border

The problem of the "wrapped" panel is fixed for me on the latest suse rpm (kde
4.0.3 release 20.2)!! Now the panel is displayed correctly even when size is
set to "tiny"!
There's only a "small visual defect":as you can see on the attached screenshot
on the tray bar if you have some icons of kde3 application (in my case
kbluetooth, kmix and knetworkmanager) the upper border of the panel is not
displayed

this is still not fixed in kubuntu with kde 4.0.4. just about nearly fits on Small but Tiny messes up the whole panel bar

It's fixed in trunk/ and should be part of 4.1-beta1. Can someone close this bug? I seem to lack karma.

*** Bug has been marked as fixed ***.

This bug has not been fixed.

The clock seems to render nicely in r819867.

This was reported against the 4.0 BRANCH and although it appears to have been fixed in TRUNK, the fix has not been backported to the BRANCH.  Should we close this any how, or can the fix be backported to the 4.0 BRANCH?

Since there wasn't any one commit that explicitly fixed the problem (to the best of my knowledge) and due to Widgets on Canvas a backport would entail porting the WoC'd clock applet back to the 4.0 branch. I doubt this is going to happen. They've been closing 4.0 bugs that are fixed in 4.1 anyway, so this should be resolved fixed.

Since it can't be easily backported, I am closing it.

Yes, given that 4.1 is just around the corner, we'd like development efforts to now focus on making that release as solid as possible and some of these backports are pretty non-trivial.

I'm absolutely not against someone doing the backport if they want to, of course =) but it's just not a priority for the core plasma group at the moment as we prepare for 4.1.0.

I just implemented #3 from comment 6 (https://bugs.kde.org/show_bug.cgi?id=158762#c6). patch attached.

Created attachment 29078 [details]
puts date at the right of time on horizontal formfactor

I simply ignored comments #7 and #8. #7-5) should be feasible, but this first iteration just implements it for horizontal panels.

Created attachment 29081 [details]
also correctly asks for enough hspace for both time and date/timezone.

(In reply to comment #47)
> Created an attachment (id=29081) [details]
> also correctly asks for enough hspace for both time and date/timezone.
> 

Hi,
I just succesfully patched and compiled 4.1.82 and it works correctly. Date and time is in one line as in kde3.

Thanks a lot

Created attachment 29369 [details]
puts date at the right of time on horizontal formfactor only when there's no enought vertical space

asp er Kluegler petition, this patch pnly puts deta/tz besides time if there's not enough vertical space. I prefer the other implementatio (do this always), so I don't check the 'obsoletes" question.

SVN commit 901830 by aseigo:

date-beside-time-depending-on-height; based on a patch by Marcos Dione with a number of modifications.
the code in this applet is getting a little convoluted, could use a nice cleaning in places.
CCBUG:158762
BUG:178721


 M  +217 -111  clock.cpp  
 M  +5 -1      clock.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=901830

Regression:

Does not work with 4.1.96 :-(

Created attachment 30257 [details]
A Screenshot

this was committed after 4.1.96, wasn't it?

Sorry, I really don't know :-(

Well,
there was an another change at 04/01/09 and it is included in 4.1.96.
I really think it worked correctly in 4.1.87 (BETA2)

http://websvn.kde.org/trunk/KDE/kdebase/workspace/plasma/applets/digital-clock/clock.cpp?r1=901830&r2=905255

SVN commit 911269 by aseigo:

QFont returns -1 for pixelSize when set with a point size. so set with a pixel size, and now it works again.
CCBUG:158762


 M  +5 -4      clock.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=911269

SVN commit 911271 by aseigo:

QFont returns -1 for pixelSize when set with a point size. so set with a pixel size, and now it works again.
CCBUG:158762


 M  +5 -4      clock.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=911271

Thanks a lot. The panel have to be a really thin, but works :-)