1151735 - The font panel from Reader View should disappear when selecting text from the page

Status: RESOLVED FIXED

(Toolkit :: Reader Mode, defect, P4)

Description

[Andrei Vaida [:avaida]]

Note: this is a follow-up issue filed for Bug 1137211.

Reproducible on:
Nightly 40.0a1 (2015-04-06), Aurora 39.0a2 (2015-04-06), Beta 38.0b2-build1 (20150406174117)

Affected platforms:
Windows 7 (x64), Ubuntu 14.04 (x64), Mac OS X 10.9.5

Steps to reproduce:
1. Launch Firefox.
2. Open a Reader View compatible page - (e.g.) http://www.bbc.com/future/story/20150402-do-colours-really-change-our-mood
3. Click the "Enter Reader View" button from the Location Bar.
4. Click the "Type controls" (Aa) button from Reader View's controls bar.
5. Select text from the page currently displayed in Reader View, using the mouse cursor.

Expected result:
The text is selected and the font panel is dismissed.

Actual result:
The text is indeed successfully selected but the font panel remains in focus. Happens when selecting images as well.

Comment 1

[Justin Dolske [:Dolske]]

The panel disappears for me when the mousebutton is released -- it's just visible *while* selecting text.

We should be listening for mousedown, not click.

Comment 2

[sonali18317]

:Gijs, can I try solving this bug?

Comment 3

[sonali18317]

Attachment: Hide font panel when text is being selected

Comment 4

[sonali18317]

:Gijs, there is a revision that the phab-bot requested. I am new to versioning. Is hg add path/to/file, then hg commit --amend and then arc diff the right way to update the commit?

Comment 5

[:Gijs (he/him)]

(In reply to sonali18317 from comment #4)

:Gijs, there is a revision that the phab-bot requested. I am new to versioning. Is hg add path/to/file, then hg commit --amend and then arc diff the right way to update the commit?

I think you figured this out already, but just in case: you don't need the hg add, just the hg commit --amend (or hg amend), and then resubmit, yep.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3b996d54a151
Hide font panel when text is being selected r=Gijs

Comment 7

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/3b996d54a151