Closed
Bug 1048230
Opened 10 years ago
Closed 10 years ago
Global indicator inaccessible when taskbar is on top of the screen
Categories
(Firefox :: Site Permissions, defect)
Tracking
()
People
(Reporter: bmaris, Assigned: florian)
References
Details
Attachments
(1 file)
(deleted),
patch
|
enndeakin
:
review+
Sylvestre
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
Reproduced using latest Nightly 34.0a1
STR:
1. Open Firefox
2. Set media.getusermedia.screensharing.enabled to true
3. Open http://queze.net/goinfre/ff_gum_test.html in a new tab
4. Click Audio & Video
5. Click Share Selected Devices
6. Move the taskbar from down to UP
Expected result: The indicator stays above the taskbar and it can be accessed.
Actual result: The indicator stays under the taskbar and it can`t be accessed.
Notes:
1. This is Windows only, I did not manage to move the 'taskbar' on ubuntu.
2. Screenshot showing the issue: https://db.tt/BNIcJwTa
Assignee | ||
Comment 1•10 years ago
|
||
(In reply to Bogdan Maris, QA [:bogdan_maris] from comment #0)
> 1. This is Windows only, I did not manage to move the 'taskbar' on ubuntu.
bug 1041675 seems the Linux equivalent.
I wonder if the solution here wouldn't be to use screen.availTop instead of 0 when positioning the indicator.
Assignee | ||
Updated•10 years ago
|
Flags: firefox-backlog+
Assignee | ||
Comment 2•10 years ago
|
||
Attachment #8474499 -
Flags: review?(enndeakin)
Comment 3•10 years ago
|
||
Comment on attachment 8474499 [details] [diff] [review]
Display the window at y=availTop instead of 0
Seems ok. Is there a reason the popup is locked vertically? You might consider removing all the custom window dragging code and just using WindowDraggingUtils.jsm
Attachment #8474499 -
Flags: review?(enndeakin) → review+
Comment 4•10 years ago
|
||
Hi Florian, can you provide a point value and if the bug should be marked as [qa+] or [qa-].
Assignee: nobody → florian
Status: NEW → ASSIGNED
Iteration: --- → 34.2
QA Whiteboard: [qa?]
Flags: needinfo?(florian)
Assignee | ||
Comment 5•10 years ago
|
||
(In reply to Neil Deakin from comment #3)
> Is there a reason the popup is locked vertically?
Because UX (phlsa) said so.
> You might
> consider removing all the custom window dragging code and just using
> WindowDraggingUtils.jsm
We (Gijs) actually started from WindowDraggingUtils.jsm and forked it to implement the requested UX.
Points: --- → 2
QA Whiteboard: [qa?] → [qa+]
Flags: needinfo?(florian)
Assignee | ||
Updated•10 years ago
|
Whiteboard: [screensharing-uplift]
Assignee | ||
Comment 6•10 years ago
|
||
Updated•10 years ago
|
Iteration: 34.2 → 34.3
QA Whiteboard: [qa+]
Flags: qe-verify+
Comment 7•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
Reporter | ||
Comment 8•10 years ago
|
||
Verified that if a call is made with taskbar on top it the global indicator will show under the taskbar not behind it. Tested on Windows 7 64bit and Windows 8.1 Microsoft Surface PRO, using latest Nightly 34.0a1.
Status: RESOLVED → VERIFIED
Assignee | ||
Comment 9•10 years ago
|
||
Comment on attachment 8474499 [details] [diff] [review]
Display the window at y=availTop instead of 0
Approval Request Comment
[Feature/regressing bug #]: bug 1037408
[User impact if declined]: Windows users with the taskbar at the top of the screen may be unable to use the global webrtc sharing indicator that would be covered by the task bar.
[Describe test coverage new/current, TBPL]: currently on m-c, verified by QA.
[Risks and why]: Low, tiny self-contained code change.
[String/UUID change made/needed]: none.
Attachment #8474499 -
Flags: approval-mozilla-aurora?
Updated•10 years ago
|
status-firefox33:
--- → affected
status-firefox34:
--- → fixed
Updated•10 years ago
|
Attachment #8474499 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Reporter | ||
Updated•10 years ago
|
Assignee | ||
Comment 10•10 years ago
|
||
Updated•10 years ago
|
Whiteboard: [screensharing-uplift]
Reporter | ||
Comment 12•10 years ago
|
||
Also verified this using latest Aurora 33.0a2 on Windows 7 64bit and Windows 8.1 Microsoft Surface Pro.
Assignee | ||
Updated•10 years ago
|
Component: WebRTC → Device Permissions
Product: Core → Firefox
Target Milestone: mozilla34 → Firefox 34
You need to log in
before you can comment on or make changes to this bug.
Description
•