Bug 155010 - plasmoid icon resize weirdness
Summary: plasmoid icon resize weirdness
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-01-03 10:09 UTC by Shieldfire
Modified: 2008-01-03 14:22 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Shieldfire 2008-01-03 10:09:28 UTC
Version:           rc2++ (using KDE KDE 3.97.0)
Installed from:    Unlisted Binary Package
OS:                Linux

When enlarging desktop icons there doesn't seem to be any upper limit of the size. Which means you can have a icon filling the entire desktop. Not good.
Comment 1 Marco Martin 2008-01-03 13:38:52 UTC
SVN commit 756574 by mart:

make icons at most 256 pixels

BUG: 155010

 M  +1 -0      icon.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=756574
Comment 2 Jason Stubbs 2008-01-03 13:53:14 UTC
Why isn't this good? Why should there be an arbitrary limit on how big an icon should be?
Comment 3 Marco Martin 2008-01-03 13:58:19 UTC
maybe set it even bigger than 256, but at the moment the main reason i think that scaling it beyond a certain size takes an insane amount of cpu and nearly freezes plasma, at least on a not so powerful cpu
Comment 4 Jason Stubbs 2008-01-03 14:13:08 UTC
Painting at the new size (and thus the large cpu usage) only happens after the user releases the mouse button though, no? Resizing it smaller again shouldn't be difficult. Another way to get around the CPU usage would be to ensure that the graphical icon doesn't go beyond a certain size (which would make sense especially for raster graphics) but allow the applet to be resized so that the user can make more text visible.

The minimum screen size is 800x600 and according to wikipedia the current largest DVI resolution is 3840x2400. 256 pixels is therefore somewhere between 7% and 32% of the screen width. Choosing any other value will still have a huge variance what the visible maximum turns out to be...

I won't push this, but setting a maximum size for no good reason doesn't really seem right. And if there is a good reason, it's likely general to all applets and should probably dealt with in plasma.
Comment 5 Aaron J. Seigo 2008-01-03 14:16:55 UTC
yes, this commit as well as this bug report are completely bogus. this really is a matter of "well, if you don't want an icon that big, don't make them that big!" limiting others in this way is not Plasma(tm).
Comment 6 Aaron J. Seigo 2008-01-03 14:20:40 UTC
SVN commit 756606 by aseigo:

arbitrary upper limits are probably not a good idea; just don't make them so big if you won't want them that big. as for cpu usage, even HUUUGE icons on my 1.8Ghz machine don't even make plasma cough. i'm sure with complex enough svg icons and a slow enough processor it would have a rendering impact but, again, just don't make them so frigging huge then.
CCBUG:155010


 M  +0 -1      icon.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=756606
Comment 7 Marco Martin 2008-01-03 14:22:47 UTC
ok, nevermind :)