Bug 12698 - Text entry - Selected character not centered vertically in input bar
: Text entry - Selected character not centered vertically in input bar
Status: CLOSED FIXED
Product: SqueezePlay
Classification: Unclassified
Component: UI Input
: unspecified
: PC Windows Vista
: P1 major (vote)
: 7.4.0
Assigned To: Wadzinski Tom
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-07-06 13:26 UTC by ndijulio
Modified: 2009-10-05 14:31 UTC (History)
3 users (show)

See Also:
Category: ---


Attachments
screenshot from campfire discussing this issue (303.13 KB, image/png)
2009-07-06 13:35 UTC, Ben Klaas
Details

Note You need to log in before you can comment on or make changes to this bug.
Description ndijulio 2009-07-06 13:26:16 UTC
We need a method of centering characters/text strings based on a fix height asset area so that no matter what the actual character height (: or A) is centered vertically.
Comment 1 Ben Klaas 2009-07-06 13:35:15 UTC
where would an underscore character be rendered vertically using such a method?
Comment 2 Ben Klaas 2009-07-06 13:35:53 UTC
Created attachment 5410 [details]
screenshot from campfire discussing this issue
Comment 3 Blackketter Dean 2009-07-06 16:45:52 UTC
I think the baseline of the character needs to be fixed at a specific  
position, and then make sure the character set fits at that size,   
trying to dynamically center vertically will look weird and end up  
being a losing battle.


On Jul 6, 2009, at 1:35 PM, bugs@bugs.slimdevices.com wrote:

> https://bugs-archive.lyrion.org/show_bug.cgi?id=12698
>
>
>
>
>
> --- Comment #2 from Ben Klaas <bklaas@slimdevices.com>  2009-07-06  
> 13:35:53 ---
> Created an attachment (id=5410)
> --> (https://bugs-archive.lyrion.org/attachment.cgi?id=5410)
> screenshot from campfire discussing this issue
>
> -- 
> Configure bugmail: https://bugs-archive.lyrion.org/userprefs.cgi?tab=email
> ------- You are receiving this mail because: -------
> You are on the CC list for the bug.
Comment 4 ndijulio 2009-07-06 19:12:36 UTC
Agreed.  I think we just need to expand the input text window to 46px high (which is done i believe) and adjust the spacing a little...
Comment 5 Ben Klaas 2009-07-07 20:54:55 UTC
Tom, I'm passing this one over to you, because I think what's being asked is more at the C level. I did a little experimenting with charOffsetY in the skin and what we have it set at now is about as good as it's going to get without some underlying changes to how the positioning works.
Comment 6 SVN Bot 2009-08-14 11:34:17 UTC
 == Auto-comment from SVN commit #7086 to the jive repo by tom ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7086 ==

Fixed Bug:12698
Description:
- char under cursor down one pixel for better centering across various chars
- input window is 46 px high
- lowest char on wheel top section was too close to input window area, move up 6 pixels (using new param wheelCharOffsetY)
Comment 7 James Richardson 2009-10-05 14:31:06 UTC
This bug has been marked as fixed in the 7.4.0 release version of SqueezeBox Server!
    * SqueezeCenter: 28672
    * Squeezebox 2 and 3: 130
    * Transporter: 80
    * Receiver: 65
    * Boom: 50
    * Controller: 7790
    * Radio: 7790  

Please see the Release Notes for all the details: http://wiki.slimdevices.com/index.php/Release_Notes

If you haven't already, please download and install the new version from http://www.logitechsqueezebox.com/support/download-squeezebox-server.html

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.