885471 - Soft home button isn't enabled by default on Nexus 4

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::System, defect)

Comment 1

[(inactive after 6/18) Alive Kuo [:alive]]

Hmm...something wrong in the CSS rule?

Comment 2

[Michael Wu [:mwu]]

David Baron suggested "@media not all and (-moz-physical-home-button)"

I tried "@media not (-moz-physical-home-button)" while testing the css patch and that didn't work. David's suggestion did, however.

Comment 3

[(inactive after 6/18) Alive Kuo [:alive]]

OK...I had tested it but worked, strange.

I think I also forgot to fire holdhome event.

I will fix bug 885473 as well.

Comment 4

[(inactive after 6/18) Alive Kuo [:alive]]

I have one question...if I fixed this, will people using simulator see it default turn on?
I think simulator doesn't have "hardware" button right?

Comment 5

[Michael Wu [:mwu]]

Yeah, it'll turn on on desktop. The simulator has its own "hardware" button, but that can be removed once we have this one. Alternately, they might want to force it off. Let's see if myk has any concerns here.

Comment 6

[(inactive after 6/18) Alive Kuo [:alive]]

(In reply to Michael Wu [:mwu] from comment #5)
> Yeah, it'll turn on on desktop. The simulator has its own "hardware" button,
> but that can be removed once we have this one

If you mean to replace it with this, I don't think so...
The problem is every app's size would be cut by 5rem if we're using software button.
It's not a big deal on nexus device because it's has high resolution.
But will cause trouble on simulator I think..

In firefox nightly, I could use window.location.protocol to detect and turn off natively I think.
But currently I have no idea if gaia was running on simulator or not..

. Alternately, they might want
> to force it off. Let's see if myk has any concerns here.

Comment 7

[(inactive after 6/18) Alive Kuo [:alive]]

Attachment: https://github.com/mozilla-b2g/gaia/pull/10525

patch v1:
1. Remove the confusing css style.
2. Add holdhome event on software button.
3. Use correct media query rules.
4. Fix simulator and nightly issues. Default turn the software buttons off on them.

Comment 8

[(inactive after 6/18) Alive Kuo [:alive]]

With the patch we don't need to change anything in simulator now.

Comment 9

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 765786 [details]
https://github.com/mozilla-b2g/gaia/pull/10525

My nexus 4 is not handy now, :mwu could you test this?

Comment 10

[Michael Wu [:mwu]]

Comment on attachment 765786 [details]
https://github.com/mozilla-b2g/gaia/pull/10525

This definitely works better. The only weird thing now is that dragging from the home button still fires an event.

Comment 11

[(inactive after 6/18) Alive Kuo [:alive]]

(In reply to Michael Wu [:mwu] from comment #10)
> Comment on attachment 765786 [details]
> https://github.com/mozilla-b2g/gaia/pull/10525
> 
> This definitely works better. The only weird thing now is that dragging from
> the home button still fires an event.

Does that really matter? I see the same thing on Firefox Nightly. IMO the only weird thing is that we cannot get UX love on the button...

BTW I change the pull request a bit for something is still wrong on desktop. Would you mind trying again on real device? And it's not able to turn off software button on the device really without hardware button now so the settings should be removed..

Comment 12

[Michael Wu [:mwu]]

(In reply to Alive Kuo [:alive] from comment #11)
> Does that really matter? I see the same thing on Firefox Nightly. IMO the
> only weird thing is that we cannot get UX love on the button...
> 

Don't know if it matters. This is a lot better than the bugs before so I'll take this trade.

> BTW I change the pull request a bit for something is still wrong on desktop.
> Would you mind trying again on real device? And it's not able to turn off
> software button on the device really without hardware button now so the
> settings should be removed..

Still seems to work ok here.

Comment 13

[(inactive after 6/18) Alive Kuo [:alive]]

(In reply to Michael Wu [:mwu] from comment #12)
> (In reply to Alive Kuo [:alive] from comment #11)
> > Does that really matter? I see the same thing on Firefox Nightly. IMO the
> > only weird thing is that we cannot get UX love on the button...
> > 
> 
> Don't know if it matters. This is a lot better than the bugs before so I'll
> take this trade.
> 
> > BTW I change the pull request a bit for something is still wrong on desktop.
> > Would you mind trying again on real device? And it's not able to turn off
> > software button on the device really without hardware button now so the
> > settings should be removed..
> 
> Still seems to work ok here.
Thanks, merging.

Comment 14

[(inactive after 6/18) Alive Kuo [:alive]]

master
https://github.com/mozilla-b2g/gaia/commit/2f4bef9795364def95629f894a117a09809513d3