943400 - Don't invoke the text-selection action-bar on initial input focus

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Aaron Train [:aaronmt]]

Regression from bug 768667. On focus the action-bar is shown on-top of the nav-bar. On key-insert, the action-bar is removed.

Focus an <input/>, see action-bar. We should wait for a selection.

Comment 1

[Wesley Johnston (:wesj)]

This isn't a regression. We didn't have these before.

I also don't think this should ONLY happen on selection. We need a way to paste even if there is not text in the input (and we have other options like "Add search engine" as well). Android uses a floating "Paste" button that shows up randomly sometimes. I hate it, and I'd hate to copy awful UI design (but I have a feeling we will end up doing it anyway, and then regretting it when Android removes it later).

Part of the problem here is that the cursor disappears when you start typing, which is non-Androidy as well. When the cursor disappears, we stop showing the action mode.

Alternatively, we could try something like only showing if we're focusing an already focused input. But I don't think only showing the bar when something is selected will work.

Comment 2

[Wesley Johnston (:wesj)]

I talked to lucas about this for a bit. So, IMO, the problem here is that we're showing a selection handle whenever a textbox is focused. Java (currently) uses that (along with the presence of actions that can be performed) as a key to say "Show an actionbar". Typically, we've been moving away from js driving the java ui, but text-selection is one case where... that's going to be tough. JS has to know a lot about the java ui to tell it what type of handle to show and where.

So in this case, since JS is driving the UI, maybe we just let JS be smart and not show the handle when the textbox is first focused (or if we're really really in love with showing the handle then, we send zero actions with it). Will throw up a test build.

Comment 3

[Wesley Johnston (:wesj)]

Attachment: Patch

This implements that:
http://people.mozilla.com/~wjohnston/actionmode2.apk

What do you think ian?

Comment 4

[Ian Barlow (:ibarlow)]

This feels pretty good, Wes. I wasn't convinced on using the action bar for the paste command as well (vs that floating menu, which I actually like) but it feels good in practice, so let's go with it. 

I have some overall visual feedback for you re: action mode in general, but I'll file that in a separate bug.

Comment 5

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 8338658 [details] [diff] [review]
Patch

>+            // Was the element already focused before it was clicked?
>+            let isFocused = element == BrowserApp.getFocusedInput(BrowserApp.selectedBrowser, true);

Use () around the test:
              let isFocused = (element == BrowserApp.getFocusedInput(BrowserApp.selectedBrowser, true));

Comment 6

[Wesley Johnston (:wesj)]

https://hg.mozilla.org/integration/fx-team/rev/5d90f8c198bf

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/5d90f8c198bf