928592 - BrowserToolbar back button should dismiss awesomescreen on tablet

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: Awesomescreen, defect)

Description

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

1) Open a web page
2) Click a link
3) Click the URL bar to open the awesomescreen
4) Hit back on the browser toolbar

Expected: The awesomescreen is dismissed
Actual: The previous page is returned to in the web content, but this change is not visible because the awesomescreen is still present

Ian, is "actual" the expected behavior?

Comment 1

[Gabriela [:gaby2300]]

I cannot reproduce this bug using my Galaxy 10'1 Tab and Android 3.1
After step 4 the previous page is returned to and I can see its content.

Comment 2

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Just to be clear, are you hitting back in the Firefox UI (as opposed to Android's back button)?

Comment 3

[Gabriela [:gaby2300]]

Yes, I am tapping the back Firefox UI button.

Comment 4

[Lucas Rocha (:lucasr)]

This will be fixed by bug 918007. We'll disable all other toolbar elements while in editing mode. Closing this one.

Comment 5

[Lucas Rocha (:lucasr)]

I closed as WONTFIX just to make sure that no one would work on this before bug 918007 is fixed. But maybe adding a dependency is a better way to handle this.

Comment 6

[Ian Barlow (:ibarlow)]

(In reply to Michael Comella (:mcomella) from comment #0)
> 1) Open a web page
> 2) Click a link
> 3) Click the URL bar to open the awesomescreen
> 4) Hit back on the browser toolbar
> 
> Expected: The awesomescreen is dismissed
> Actual: The previous page is returned to in the web content, but this change
> is not visible because the awesomescreen is still present
> 
> Ian, is "actual" the expected behavior?

un-needinfoing due to bug 918007

Comment 7

[Lucas Rocha (:lucasr)]

The patches for bug 918007 have landed in Nightly. You cannot use the Back button while editing anymore. Closing.

Comment 8

[Lucas Rocha (:lucasr)]

Lukas, isn't it more like this is fixed/verified in Fx 26/27?

Comment 9

[Lukas Blakk [:lsblakk] use ?needinfo]

(In reply to Lucas Rocha (:lucasr) from comment #8)
> Lukas, isn't it more like this is fixed/verified in Fx 26/27?

well it was marked as fixed/verified in but 918007, so there's nothing to track for landing or uplift in this bug.

Comment 10

[Lukas Blakk [:lsblakk] use ?needinfo]

but it's no matter to me, putting to fixed if that makes more sense to others - I just care about the tracking status of the bug.