#8662 NORM 8.2.0 (: xo man jumps around while zooming

[Zarro Boogs per Child]

#8662: xo man jumps around while zooming
---------------------------+------------------------------------------------
   Reporter:  cscott       |       Owner:  cscott              
       Type:  enhancement  |      Status:  new                 
   Priority:  normal       |   Milestone:  8.2.0 (was Update.2)
  Component:  sugar        |     Version:  not specified       
 Resolution:               |    Keywords:  polish:8.2.0? r+    
Next_action:  review       |    Verified:  0                   
  Blockedby:               |    Blocking:                      
---------------------------+------------------------------------------------
Changes (by cscott):

  * keywords:  polish:8.2.0? r? => polish:8.2.0? r+


Comment:

 From irc:
 {{{
 (02:22:21 PM) eben: marcopg_: Do you have any idea how you "fixed" :-P it
 before?  There was an old ticket for this marked fixed.
 (02:23:02 PM) marcopg_: eben: I suspect it might be grid rounding, not
 sure though, I'd have to read the code
 (02:23:16 PM) eben: c_scott, marcopg_:  For the record, the zoom will be
 off in the Home view when set to freeform anyway, because the XO is forced
 onto the 4px grid, and not exactly centered....
 (02:23:25 PM) eben: We shouldn't likely deal with that somehow too...not
 sure.
 (02:23:26 PM) C. Scott Ananian: marcopg_: i tried rounding, and it didn't
 make a change.  the rounding might happen elsewhere, though
 (02:23:30 PM) eben: (not now, of course)
 (02:23:40 PM) eben: marcopg_: no grid for ring view!
 (02:23:51 PM) marcopg_: eben: ah
 (02:23:53 PM) eben: I know the grid breaks it in freeform
 (02:24:06 PM) marcopg_: eben: maybe toolbar size then?
 (02:24:12 PM) C. Scott Ananian: eben: no, the animation still looks right
 in grid view
 (02:24:21 PM) eben: marcopg_: maybe...but why would it be off in both
 axes?
 (02:24:22 PM) marcopg_: I think the centering code needs to take in
 account the toolbar size
 (02:24:25 PM) C. Scott Ananian: eben: we actually seem to use the grid
 layout for friends view, and maybe mesh view as well.
 (02:24:42 PM) marcopg_: eben: well yeah, that's weird :)
 (02:24:54 PM) eben: ohhhh....right.  Indeed.
 (02:24:58 PM) eben: It's not off in Home..
 (02:25:03 PM) eben: only in groups and neighborhood.
 (02:25:05 PM) eben: it's the grid
 (02:25:10 PM) C. Scott Ananian: anyway, re 8662: it was deliberately an
 attempt at the smallest possible patch for our specific build, i can apply
 the patch only in pilgrim if that's easiest.
 (02:25:25 PM) C. Scott Ananian: i just don't want to do so w/o the consent
 of the sugar devs =)
 (02:25:30 PM) eben: It forces alignment on 4x4px blocks
 (02:25:34 PM) marcopg_: I can apply it in the rpm
 (02:25:46 PM) C. Scott Ananian: eben: if it's the grid, then the
 correction should be cross-platform.
 (02:25:57 PM) eben: c_scott: right.  it will be.
 (02:26:03 PM) eben: I understand it now, so I'm content. :)
 (02:26:07 PM) C. Scott Ananian: marcopg_: does that make your life easier?
 (02:26:15 PM) eben: (Still needs a better fix, but the hack is acceptable)
 (02:26:41 PM) marcopg_: c_scott: no :)
 (02:26:52 PM) C. Scott Ananian: well, maybe not cross platform: it depends
 on how the native screen size fits into a 4x4 grid.
 (02:27:09 PM) marcopg_: if you apply it in pilgrim it's easier for me,
 just didn't want to make your life harder :)
 (02:27:53 PM) eben: c_scott: true....if we used a grid size of 2 we'd be
 fine.
 (02:27:59 PM) C. Scott Ananian: marcopg_: ok, so what's your preference?
 rpm, sucrose-0.82 branch, or pilgrim?
 (02:28:49 PM) marcopg_: bah
 (02:28:51 PM) eben: c_scott: so there's a 50% chance that it works on any
 given platform.  Well.....better....actually...given standard screen
 sizes.
 (02:28:55 PM) marcopg_: let's use sucrose-0.82
 (02:29:06 PM) marcopg_: it can't make things worst on other screens
 (02:29:33 PM) C. Scott Ananian: marcopg_: that makes my life easier =)
 (02:30:11 PM) marcopg_: c_scott: so r+ on that one
 }}}

-- 
Ticket URL: <http://dev.laptop.org/ticket/8662#comment:2>
One Laptop Per Child <http://laptop.org/>
OLPC bug tracking system