Picture zooming annoyances

[Art]

don't get me wrong, this is a great feature of xbmc, but i'd like to report a few annoyances to improve it:
first the banner displaying the zoom ratio, it is totally unnecessary, showing it in the corner of the screen would do the trick imho. if it has to stay, it shouldn't overlap the picture beneath it, in fact, you should be able to pan entirely the picture.
one thing that annoys me too, is the minimum zoom ration that is set to 2x, would it be possible to set it to 1.5x ?

what do you guys think ?

art

[jmarshall]

i believe you can change the banner when zooming thing by changing the skin.

the zoom amounts i agree with. perhaps you could start a poll and see if there is some consensus among users? post say 4-5 different ranges and see if we can come up with something that most people agree on.

[thugg]

maybe also being able to skip to the next picture while zoomed. it's not like it's all that hard to zoom out, but you shouldn't need to.

[mosax]

some of this suggestions and more in here (http://www.xboxmediaplayer.de/cgi-bin/forums/ikonboard.pl?act=st;f=4;t=1769;hl=zoom).

[Rocky]

the next-picture when zoomed-in problem and info display annoyance are easy to fix:

this allows next/prev picture step when zoomed (next picture is un-zoomed); and you must press the info button to get the banner disaply:

i also noticed that the info display starts counting at zero, instead of one for the current picture of pictures numbers:


index: guiwindowslideshow.cpp
================================================== =================
rcs file: /cvsroot/xbmc/xbmc/xbmc/guiwindowslideshow.cpp,v
retrieving revision 1.15
diff -r1.15 guiwindowslideshow.cpp
1c1
<
---
>
330c330
< if (!m_bshowinfo && m_izoomfactor == 1)
---
> if (!m_bshowinfo)
362c362
< strslideinfo.format("%i/%i", m_icurrentslide ,m_vecslides.size());
---
> strslideinfo.format("%i/%i", m_icurrentslide+1 ,m_vecslides.size());
586c586
< if (m_izoomfactor==1)
---
> //if (m_izoomfactor==1)
590c590
< if (m_izoomfactor==1)
---
> //if (m_izoomfactor==1)
678,679c678,679
< float fx=2*action.famount1;
< float fy=2*action.famount2;
---
> float fx=20*action.famount1;
> float fy=20*action.famount2;