915279 - Selection monocles position is incorrect after zoom

Status: RESOLVED DUPLICATE of bug 940952

(Firefox for Metro Graveyard :: Input, defect)

Description

[Jim Mathies [:jimm]]

STR:

1) open a page with some text
2) select some text
3) zoom in a bit

Comment 1

[Kartikaya Gupta (email:[e-mail])]

How are the selection monocles drawn in Metro? Are they a thing in Gecko or are they overlaid using Windows APIs on top? It looks to me like they are drawn using Gecko so if you make them position:absolute relative to the content document they should stay in place when you pan around.

I'm also CC'ing Ehsan because we were discussing text selection and how to do it consistently across all platforms last week, and he might be interested in knowing what's happening here.

Comment 2

[Jim Mathies [:jimm]]

(In reply to Kartikaya Gupta (email:[e-mail]) from comment #1)
> How are the selection monocles drawn in Metro? Are they a thing in Gecko or
> are they overlaid using Windows APIs on top? It looks to me like they are
> drawn using Gecko so if you make them position:absolute relative to the
> content document they should stay in place when you pan around.
> 
> I'm also CC'ing Ehsan because we were discussing text selection and how to
> do it consistently across all platforms last week, and he might be
> interested in knowing what's happening here.

This is a front end issue which is where we manage the monocles. We need to hide them at the start of a pan or zoom and reposition/display when the apz operation is complete.

Comment 3

[(no longer active)]

(In reply to Kartikaya Gupta (email:[e-mail]) from comment #1)
> How are the selection monocles drawn in Metro? Are they a thing in Gecko or
> are they overlaid using Windows APIs on top? It looks to me like they are
> drawn using Gecko so if you make them position:absolute relative to the
> content document they should stay in place when you pan around.

How would that work?  Zooming should change the coordinates used on the position:absolute element.

> I'm also CC'ing Ehsan because we were discussing text selection and how to
> do it consistently across all platforms last week, and he might be
> interested in knowing what's happening here.

I am, yes!

Comment 4

[Jim Mathies [:jimm]]

(In reply to :Ehsan Akhgari (needinfo? me!) from comment #3)
> (In reply to Kartikaya Gupta (email:[e-mail]) from comment #1)
> > How are the selection monocles drawn in Metro? Are they a thing in Gecko or
> > are they overlaid using Windows APIs on top? It looks to me like they are
> > drawn using Gecko so if you make them position:absolute relative to the
> > content document they should stay in place when you pan around.
> 
> How would that work?  Zooming should change the coordinates used on the
> position:absolute element.
> 
I would hope that coords in text ranges would update accordingly after the apz transforms due to a zoom or pan. If not, we have a another problem on our hands.

Comment 5

[Jim Mathies [:jimm]]

So panning works ok since the the monocles are positioned at screen coords, and after panning the selection text rect appears to be correct. I can pan the page and then select and the monocles are positioned correctly. I'm guessing this is due to the content scroll position updating we do when we move the display port around. 

With zoomed content though the coords are wrong. We can probably skip around this by disabling selection while zoomed for now. That kind of sucks but if there's no easy fix we may have to wait until all the selection code is down in platform.

Comment 7

[Jim Mathies [:jimm]]

Attachment: wip

Zoom is a little messed up currently making this hard to test. Posting a wip that kinda works.