Bug 160640

Summary:	position of preview wallpaper
Product:	[Unmaintained] plasma4	Reporter:	sts
Component:	general	Assignee:	Plasma Bugs List <plasma-bugs>
Status:	RESOLVED FIXED
Severity:	normal	CC:	finex
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	Screenshot wallpaper selection of center, with huge fonts

Description sts 2008-04-10 09:03:45 UTC

Version:            (using Devel)
Installed from:    Compiled sources

If I want to change my wallpaper I can see the preview of wallpaper isn't in the right position.

see attachment for more.

On another computer all looks good.

My system: thinkpad a31p, radeon mobility fire gl 7800, 1600x1200

Comment 1 sts 2008-04-10 09:04:15 UTC

Created attachment 24285 [details]
Screenshot

Comment 2 FiNeX 2008-04-12 00:41:51 UTC

Exactly on what revision? Thanks!

Comment 3 sts 2008-05-07 13:57:28 UTC

with all :) here at the moment 4.0.71 I see this problem on my notebooks.
1. with 1600x1200 and 132x132 dpi
2. with 1400x900 and 127x127dpi

Comment 4 sts 2008-05-10 22:24:20 UTC

aah I have found the reason. You can reproduce it with a bigger font-size in kde settings. So the position of the wallpaper depends on the font-size.. :)

Comment 5 Jason Stubbs 2008-06-01 08:30:06 UTC

This appears to be fixed. Please reopen if it is still an issue for you.

Comment 6 sts 2008-06-20 10:48:24 UTC

it's still exist with 4.0.82

Comment 7 Alex Merry 2008-07-05 12:25:16 UTC

I can't reproduce this.  Do you still get it in the 4.1 betas or trunk?

Comment 8 Mario Oschwald 2008-07-10 22:53:24 UTC

I can confirm this bug with the current trunk - see attached screenshot

I think it has to do with the lenght of the new wallpaper button. Because in german the words are longer, it was probably noticed earlier by the op.
In english I had to increase the font size a lot to trigger it.

Comment 9 Mario Oschwald 2008-07-10 22:54:05 UTC

Created attachment 26025 [details]
wallpaper selection of center, with huge fonts

Comment 10 Anne-Marie Mahfouf 2008-07-13 15:34:49 UTC

SVN commit 831864 by annma:

might fix position of wallpaper on Preview screen when having big fonts or long strings - not totally sure about it as I could not reproduce the same as on the screenshots here so not closing the report.
CCBUG=160640


 M  +61 -28    BackgroundDialog.ui  


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

Comment 11 Anne-Marie Mahfouf 2008-07-13 15:37:25 UTC

Can you test it either by building trunk or by getting
http://websvn.kde.org/*checkout*/trunk/KDE/kdebase/workspace/plasma/containments/desktop/BackgroundDialog.ui?revision=831864
and rebuilding your current revision then restart plasma.

Thanks in advance!

Comment 12 Anne-Marie Mahfouf 2008-07-15 09:39:26 UTC

Nobody can have a go at testing my fix please so I can backport to 4.1 if it works? Thanks in advance!

Comment 13 Mario Oschwald 2008-07-15 13:42:01 UTC

I tested it with your fix in trunk and the image is now always correctly centered.

Comment 14 Anne-Marie Mahfouf 2008-07-15 17:00:18 UTC

SVN commit 832818 by annma:

backport r831864, thanks Mario for testing!
BUG=160640


 M  +61 -28    BackgroundDialog.ui  


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