but right now only a few people is reporting bugs.. that's sad
Maybe because only a few people are experiencing bugs, or perceive something as bug?
On my camera everything OSD related works just fine.
The filebrowser font hast hat black outline, but actually i like it. I was the one picking out that patch from the thread, but i honestly didnt read the thread very carefully - i just picked the latest patch. And after some tinkering around, it worked with my cam.
Really?
Have you set your OSD background to "transparent", color = 0x0 and turned on the clock with seconds
or set the battery to report voltage?
Is the int_incr multiplier value stuff working for you?
The problem I see is that when OSD characters are drawn, when the background is transparent, it leaves whatever was there
previously. This means that the prior character is not erased but overwritten.
After a few updates, the character will no longer be readable.
In my opinion, while it does look "nice", "cool", or "better", to have the transparent background
that allows OSD text to be closer to other camera text/icons, the current character draw code doesn't work well enough use.
I think that this could be made to work, but I don't think it will be as simple as what the current code is doing.
Adding in the check for shutter half pressed isn't a solution. It merely disables the transparent background feature
for any OSD text written when the shutter
is not pressed so you lose the "better" looking visual that the transparent background code was trying to enable.
The half pressed shutter patch solution merely makes it "work" better, when the shutter is not pressed.
It makes it work like it used to work before this transparent background capability was added.
The problem shows up again as soon as you press and hold the shutter button half way down.
Often it may not be seen because there can be a lot of stuff going on on the screen and things are being overlapped or
the screen is being repainted in the case of zebra modes, but it is is still there.
Disable, zebra and enable the clock with seconds and you will see the seconds digits overwrite each other
when the shutter is half pressed.
Also, play with the int_incr stuff. (press the zoom lever to set the integer increment values up/down).
This code will not erase the old text so that you will only see the values increment.
Using a transparent OSD background did work prior to this update.
The transparent background would erase the other text/icons if it overlapped it but it didn't
have all the issues with creating unreadable characters and breaking the int_incr feature as well.
Again, I think that this capability is really cool and can be made to work for OSD text,
but I believe that to do it right will not be easy and requires a more general solution than
what was being done in the draw_char() function.
So my feeling is that since this new transparent feature doesn't really fully work,
lets pull it out until it can be made to work properly.
--- bill
