Bug 250907 - Customizable icons size in entry view (when selected all list)
Summary: Customizable icons size in entry view (when selected all list)
Status: RESOLVED FIXED
Alias: None
Product: tellico
Classification: Applications
Component: general (show other bugs)
Version: 2.3
Platform: Gentoo Packages Linux
: NOR wishlist
Target Milestone: ---
Assignee: Robby Stephenson
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-11 19:09 UTC by Fest
Modified: 2011-05-08 00:15 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Changing maximum icon size from 128 to 256. (1.20 KB, patch)
2010-09-12 18:28 UTC, Fest
Details
PATCH: adds gui controls to the status bar to allow the user to change the icon sizes in the icon list view (8.83 KB, patch)
2011-05-07 16:13 UTC, PC LX
Details

Note You need to log in before you can comment on or make changes to this bug.
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