aerostitch / testnavit

0 stars 0 forks source link

winCE onscreen icons far too small. no response to w="nn" h="mm" #203

Open aerostitch opened 11 years ago

aerostitch commented 11 years ago

Issue migrated from trac ticket # 1166

component: core | priority: minor | keywords: winCE, OSD, dimension, portability

2013-09-22 08:12:55: law_ence.dev@ntlworld.com created the issue


svn-5643 although this happened earlier.

<osd enabled="yes" type="button" x="-50" y="-90" w="30%" h="30%" command="zoom_in()" src="zoom_in.png"/>

is my current test in navit.xml.

The w and h parameters have no effect. The (+) icon used to be a reasonable size with a default setting: neither w or h specified. Now the icon is tiny so barely visible. Which makes navit pretty much useless :-)

Same with the (-) zoom_out.png icon.

I see that there is a new section with name='Settings Layouts' in navit.xml which I have yet to explore. Perhaps that is relevant?

aerostitch commented 11 years ago

2013-09-22 10:33:57: usul changed priority from major to minor

aerostitch commented 11 years ago

2013-09-22 10:33:57: usul edited the issue description

aerostitch commented 11 years ago

2013-09-22 10:33:57: usul

aerostitch commented 11 years ago

2013-09-22 10:33:57: usul commented


Is this just about % dimension params? Or that width doesn't work at all?

I think we have already a ticket for the relative width bug, I will try to find it again...

aerostitch commented 11 years ago

2013-09-23 06:34:54: law_ence.dev@ntlworld.com commented


No, the problem is not just with %/relative values.

I tested with w="30", h="30" and also with w="100" & h="100". No change: still a tiny icon hardly visible when viewed from a distance as when driving.

aerostitch commented 11 years ago

2013-09-27 06:52:32: law_ence.dev@ntlworld.com commented


There is a very simple work around: change

src="zoom_in.png" to (say) src="zoom_in_48_48.png".

Maybe the bug is in the documentation for winCE? I would have done this much earlier if I had not been expecting the w and h parameters to work. Or maybe they should?

ael

aerostitch commented 11 years ago

2013-09-27 13:44:17: @sleske commented


That probably means your Navit package was built using the autotools build. With the autotools build, the PNG files without size specification are 8x8 pixels. So zoom_in.png will be 8x8, while zoom_in_48_48.png will be 48x48 pixels. The CMake builds use a default of 48 pixels, which is probably more reasonable.

The image handling code was changed a few times, so it may have worked earlier, but right now zoom_in.png will give you an 8x8 icon. So using src="zoom_in_48_48.png" is the right solution.

Of course this is still a bug, because Navit silently ignores the width/height setting. So please keep this ticket open. Maybe I'll find time to look into it.

aerostitch commented 11 years ago

2013-10-14 08:05:45: usul changed severity from * to normal*

aerostitch commented 11 years ago

2013-10-14 08:05:45: usul commented


Not sure, but the Maemo build just offers 96x96 icons, but I want to be able to have bigger OSD (zoom buttons here) by zooming them. New ticket?

aerostitch commented 11 years ago

2013-10-14 11:56:41: tryagain commented


usul, if you're not satisfied with following explanation, please open another ticket.

Actually, solution might be as simple as specifying configuration time options to get needed prescaled image sizes.

BTW, image size on most platforms can be adjusted by adding _w_h suffix to image name. If needed prescaled image is missing, it will be scaled on the fly.

But on Maemo, you most probably use SDL graphics plugin. It does not support zooming images on the fly at all. SDL itself seems to be missing interlnal function to do that, but there are a few add-on libraries that implement it. I'm unsure which one (if any) is present on Maemo.

aerostitch commented 11 years ago

2013-10-31 14:36:48: tryagain commented


Default image sizes should be equal to sizes specified in svgs. The fix is applied with 5709. Keeping it the ticket open as suggeseted above

aerostitch commented 6 years ago

2017-12-02 04:08:44: @jkoan

aerostitch commented 6 years ago

2017-12-02 04:08:44: @jkoan commented


This ticket was pushed back in order to bring 0.5.1 out soon.