Bug 250907

Summary:	Customizable icons size in entry view (when selected all list)
Product:	[Applications] tellico	Reporter:	Fest <fest.in>
Component:	general	Assignee:	Robby Stephenson <robby>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	kde
Priority:	NOR
Version:	2.3
Target Milestone:	---
Platform:	Gentoo Packages
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	Changing maximum icon size from 128 to 256. PATCH: adds gui controls to the status bar to allow the user to change the icon sizes in the icon list view

Description Fest 2010-09-11 19:09:34 UTC

Version:           2.3 (using KDE 4.5.1) 
OS:                Linux

I use Tellico not only to store info about my collection, but to see all my movies/books in one list and open from.
I select all movies in list (ctrl+a) and get nice list of icons. It's much easier to browse collection by icons than by reading list.

The only "but" is icons size. Imho they're pretty small. So i wanted to ask about customizable icons size (or at least bigger icons).

Reproducible: Didn't try

Comment 1 Robby Stephenson 2010-09-11 23:58:46 UTC

There's a config option for it, but no GUI at the moment. You have to edit the ~/.kde/share/config/tellicorc file.
http://docs.kde.org/development/en/extragear-office/tellico/hidden-options.html#hidden-general-options

Comment 2 Fest 2010-09-12 00:20:46 UTC

Thank you. It's really looks much prettier now. 
Still, what about extending range above 128px?

Comment 3 Fest 2010-09-12 18:28:28 UTC

Created attachment 51563 [details]
Changing maximum icon size from 128 to 256.

I'm not programmer, so not sure this patch is correct. I just changed MAX_ENTRY_ICON_SIZE to 256 in entryiconview.cpp.
Anyway it works, no crashes for half-day usage.

And look how nice covers in such size.
http://img213.imageshack.us/img213/1818/snapshot4ex.png

Comment 4 Robby Stephenson 2010-09-12 18:44:51 UTC

(In reply to comment #3)
> I'm not programmer, so not sure this patch is correct. I just changed
> MAX_ENTRY_ICON_SIZE to 256 in entryiconview.cpp.
> Anyway it works, no crashes for half-day usage.
> 
> And look how nice covers in such size.
> http://img213.imageshack.us/img213/1818/snapshot4ex.png

Nice! Your patch is fine, I'll check it in when I get back to my desktop computer. I'll update the documentation, too.

Comment 5 Robby Stephenson 2010-09-12 20:43:50 UTC

SVN commit 1174530 by rstephenson:

Increase max entry icon size to 256.

CCBUG:250907


 M  +2 -2      doc/tellico/hacking.docbook  
 M  +4 -0      tellico/ChangeLog  
 M  +1 -1      tellico/src/entryiconview.cpp  


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

Comment 6 PC LX 2011-05-07 16:11:39 UTC

I wrote a patch that adds a GUI controls (zoom in button, zoom out button, and slider) to the status bar and is visible when the icon view is visible. These controls allow the user to change the maximum icon size of the icon list view. I I also increased the maximum icon size to 512 (look nice).

Comment 7 PC LX 2011-05-07 16:13:34 UTC

Created attachment 59730 [details]
PATCH: adds gui controls to the status bar to allow the user to change the icon sizes in the icon list view

Comment 8 Robby Stephenson 2011-05-08 00:15:39 UTC

Git commit d529a55ed9794aec3ce9ee66a6edcee512f71037 by Robby Stephenson.
Committed on 08/05/2011 at 00:26.
Pushed by rstephenson into branch 'master'.

Add UI controls for changing image icon size

The maximum icon size is raised from 256 to 512.
The controls appear only when the icon view is visible.
Patch from Pedro Miguel Carvalho.
BUG: 250907

M  +4    -0    ChangeLog     
M  +1    -1    doc/hacking.docbook     
M  +0    -2    src/entryiconview.cpp     
M  +11   -1    src/entryiconview.h     
M  +52   -0    src/gui/statusbar.cpp     
M  +30   -0    src/gui/statusbar.h     
M  +9    -0    src/mainwindow.cpp     
M  +5    -0    src/mainwindow.h     

http://commits.kde.org/tellico/d529a55ed9794aec3ce9ee66a6edcee512f71037