Open Bug 1244284 Opened 4 years ago Updated 1 year ago

Drag events don't get untransformed by the resolution, if there is one

Categories

(Core :: Panning and Zooming, defect, P3)

47 Branch
defect

Tracking

()

Tracking Status
firefox47 --- wontfix
firefox48 --- wontfix
firefox49 --- wontfix
firefox50 --- wontfix
firefox51 --- fix-optional

People

(Reporter: kats, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: correctness, Whiteboard: gfx-noted, [ft:conndevices])

+++ This bug was initially created as a clone of Bug #1242690 +++

This is a follow-up for bug 1242690. There are still cases where having a resolution on desktop results in weird effects, presumably from input events not getting the callback transform applied properly. One example is with dragging text as described at https://bugzilla.mozilla.org/show_bug.cgi?id=1242690#c7
Hi :kats,
Thanks for helping this issue. 
As it's approaching code complete for Panasonic 2016 model TV.
Do you mind estimate how long would you able to fix it? 
Thank you!
Flags: needinfo?(bugmail.mozilla)
Blocks: TV_P1
I'm not sure. Bug 1242690 had to be backed out because it caused regressions on desktop, I'm trying to figure that out and get it relanded. Unfortunately I don't have an ETA - when is the code complete date?
Flags: needinfo?(bugmail.mozilla)
Keywords: correctness
Whiteboard: gfx-noted
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #2)
> I'm not sure. Bug 1242690 had to be backed out because it caused regressions
> on desktop, I'm trying to figure that out and get it relanded. Unfortunately
> I don't have an ETA - when is the code complete date?

We already passed CC date. However partner still like to see whether we can fix this before their shipment date which is around end of March.
Thanks
Flags: needinfo?(bugmail.mozilla)
Whiteboard: gfx-noted → gfx-noted, [ft:conndevices]
Ok, thanks. It's possible (but no guarantees) that we'll get this landed before the end of March but it is still a risky change so if possible I would look for other options as well.
Flags: needinfo?(bugmail.mozilla)
See Also: → 1241491
We can probably land bug 1242690 behind an #ifdef for the TV platform, because the TV platform shouldn't be affected by the scrollbar-dragging issue. That will let us continue and work on this bug (again #ifdef'd for the TV platform). Can you tell me what #ifdef I can use to isolate the TV platform? MOZ_WIDGET_GONK doesn't work for that, as we discussed previously.
Flags: needinfo?(kikuo)
Flags: needinfo?(jocheng)
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #5)
> We can probably land bug 1242690 behind an #ifdef for the TV platform,
> because the TV platform shouldn't be affected by the scrollbar-dragging

On partner's TV, they're using classic scrollbar by cursor-mode operation, so it seems not a good idea to exclude TV here.

> issue. That will let us continue and work on this bug (again #ifdef'd for
> the TV platform). Can you tell me what #ifdef I can use to isolate the TV
> platform? MOZ_WIDGET_GONK doesn't work for that, as we discussed previously.

Currently there's no #ifdef for TV platform, there were serverl discussion about this but no conclusion yet. Because now there seems still no specific TV functionality requiring such a #ifdef, and there's still unclear to treat TV as a specific platform for FxOS. (Although partner has its own #ifdef to distinguish the code for their specific platform capability)

Though
Flags: needinfo?(kikuo)
Ok so then it sounds like there isn't much we can do except try to fix the bug. Unfortunately we have limited resources here so it's unclear if we'll get to the bottom of it soon. Randall is working on the problem in bug 1242690.
Flags: needinfo?(jocheng)
My understanding is that we don't actually need this for the TV work since the interface doesn't allow dragging text around, only dragging the scrollbar. Quote from an email thread:

"In short, scrollbar dragging & hit-testing on mouse move/hover are important for TV.
The misplaced text pop-up effect is not, because currently it's forbidden on Pana's TV to select text."

We'll still want this for desktop zooming, but it's not a priority.
Assignee: bugmail.mozilla → nobody
No longer blocks: TV_P1, 1241491
Version: Trunk → 47 Branch
You need to log in before you can comment on or make changes to this bug.