Closed
Bug 1345683
Opened 8 years ago
Closed 6 years ago
[Regression] dragging tab doesn't work if it's started over sharing-icon
Categories
(Firefox :: Tabbed Browser, defect)
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: 684sigma, Unassigned)
References
Details
(Keywords: regression, ux-consistency)
Attachments
(1 file)
(deleted),
image/png
|
Details |
I have a problem with Firefox Beta 52. It doesn't happen on Firefox ESR 45.
Sometimes when I try to drag tab, it doesn't move.
It happens unpredictably, however, I noticed one specific scenario when it happens
1. Open https://mozdevs.github.io/MediaRecorder-examples/record-live-audio.html , allow to use the microphone
2. Install https://addons.mozilla.org/en-US/firefox/addon/devedition-theme-enabler/ , enable developer theme
3. Place mouse directly above the microphone favicon in the tab, hold left mouse button, move mouse horizontally
Result: Tab doesn't move
Expected: Tab should move
I see an issue when drag the webrtcIndicator (Firefox icon) above tabs bar, the moving / focus is sometimes paused. on Fx52.0 & Fx53.0b1, with/without DevEdition theme. Also occurred in Fx45.8.0esr.
I did not see the problem when dragging tab with "allow to use the microphone", and don't understand the "microphone favicon in the tab".
Has STR: --- → yes
Is it clear from attached screenshot what "microphone favicon in the tab" meant? If you don't like the wording, please suggest a better description
Can you reproduce this regression now?
Flags: needinfo?(yfdyh000)
Only appear when the tab is in background.
Blocks: 1206233
Status: UNCONFIRMED → NEW
Component: Untriaged → Tabbed Browser
Ever confirmed: true
Flags: needinfo?(yfdyh000)
Keywords: ux-consistency
Summary: Regression dragging tab doesn't work DevEdition theme sometimes → [Regression] dragging tab's sharing-icon doesn't work
Version: 52 Branch → 51 Branch
I'm dragging tab, not sharing icon.
Mozregression-gui generated this regression range:
https://hg.mozilla.org/integration/fx-team/pushloghtml?fromchange=ee2eba8cdedda9ec12caa8aa4903a525df1aaf2f&tochange=e077a9ed54589316850312a6c4460ef15472109d
->
1275262 – implement device sharing animation on background tabs
https://bugzilla.mozilla.org/show_bug.cgi?id=1275262
Has Regression Range: --- → yes
Flags: needinfo?(florian)
Keywords: regression
Summary: [Regression] dragging tab's sharing-icon doesn't work → [Regression] dragging tab doesn't work if it's started over sharing-icon
Comment 5•8 years ago
|
||
I can only reproduce when using the compact theme. Dão, do you happen to know if the compact theme does something different related to dragging tabs?
Flags: needinfo?(florian) → needinfo?(dao+bmo)
Comment 6•8 years ago
|
||
(In reply to Florian Quèze [:florian] [:flo] from comment #5)
> I can only reproduce when using the compact theme. Dão, do you happen to
> know if the compact theme does something different related to dragging tabs?
http://searchfox.org/mozilla-central/rev/624d25b2980cf5f83452b040e6d664460f9b7ec5/browser/themes/shared/compacttheme.inc.css#275-279
Flags: needinfo?(dao+bmo)
Comment 7•8 years ago
|
||
(In reply to Dão Gottwald [::dao] from comment #6)
> (In reply to Florian Quèze [:florian] [:flo] from comment #5)
> > I can only reproduce when using the compact theme. Dão, do you happen to
> > know if the compact theme does something different related to dragging tabs?
>
> http://searchfox.org/mozilla-central/rev/
> 624d25b2980cf5f83452b040e6d664460f9b7ec5/browser/themes/shared/compacttheme.
> inc.css#275-279
I guess based on this that we'd need to reset pointer-events to none for the sharing icon or something? The same might be true for other things which shouldn't be draggable.
Comment 8•6 years ago
|
||
This looks fixed to me.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•