962646 - Fx Android opens SUMO with javascript disabled from the Sync settings screen

Status: RESOLVED WONTFIX

(Firefox for Android Graveyard :: Android Sync, defect)

Description

[Ioana Chiorean]

Attachment: screenshot

Build: 2014-01-22 Nightly Firefox 29
OS: Android 4.2.2
Device: Galaxy Nexus 

Precondition: consider yourself a kid born in 2001 in January (1st-21st)

Steps:
1. Open Fennec and go to Settings ->  Sync
2. Tap 'Show me more'
3. Scroll down the page

Expected result:
- I should be able to see images for the demo how to add a device to FX Sync

Actual result:
- Images are not displayed and the hyperlinks are visible. The (non-it) user will be confused .

Comment 1

[Swarnava Sengupta (:Swarnava)]

Reproducible with

Build: Firefox 26.0
OS: Android 2.3.6
Device: Htc Desire HD


Build: Firefox 27.0 Beta
OS: Android 4.1.2
Device: Samsung Galaxy SII

Comment 2

[iamjayakumars]

Same issue in Android 4.4.1 - Samsung Galaxy S2 i9100

Comment 3

[[:philipp]]

http://mxr.mozilla.org/mozilla-release/ident?i=LINK_FIND_ADD_DEVICE&tree=mozilla-release

Comment 4

[Kadir Topal [:atopal]]

I could swear we already have a similar bug about mobile pages displaying weirdly when launched from mobile for things that are marked as relevant for mobile and desktop products. But I can't find that bug right now :/

Comment 5

[Ricky Rosario [:rrosario, :r1cky]]

We've decided to remove our <noscript/> fallback for images. Firefox removed the checkbox for disabling js and the KB is very broken without js anyway.

Grabbing in the current sprint to slamdunk it.

Comment 6

[Ricky Rosario [:rrosario, :r1cky]]

In a PR:
https://github.com/mozilla/kitsune/pull/1806

Comment 7

[Kadir Topal [:atopal]]

So, the browser disables JS when you follow the "show me more" link? Weird. That seems like a bug in Firefox, no?

Comment 8

[Ricky Rosario [:rrosario, :r1cky]]

Moving this over to Android. It is opening SUMO with javascript disabled. The SUMO knowledgebase requires javascript to work properly.

Comment 10

[Aaron Train [:aaronmt]]

It uses an Android WebView. Javascript is disabled by default.

Comment 11

[Nick Alexander :nalexander [he/him]]

Interesting.  I did not realize WebView was crippled by default.  For hysterical raisins, we used a WebView for Sync.  We're going to be loading links in Fennec for FxAccounts, will try to address this at the same time.

Comment 12

[Richard Newman [:rnewman]]

This worked on mfinkle's phone, and the fixes are all painful for other reasons, so holding off on landing this for now.

Pull request:

https://github.com/mozilla-services/android-sync/pull/402

Comment 13

[Kadir Topal [:atopal]]

(In reply to Richard Newman [:rnewman] from comment #12)
> This worked on mfinkle's phone, and the fixes are all painful for other
> reasons, so holding off on landing this for now.


Do you mean Mark didn't see the issue on his phone?

Comment 14

[Richard Newman [:rnewman]]

(In reply to Kadir Topal [:atopal] from comment #13)

> Do you mean Mark didn't see the issue on his phone?

There are two issues reported in this bug.

One is FxA "show me more" links. All links inside FxA activities now open in the browser.

The other (the title) is about SUMO display from the old Sync setup screen. SUMO displayed and behaved correctly on mfinkle's phone from that location.

Given that (a) we can't reproduce the report, and (b) we don't care about Old Sync, I'm inclined to not change this.

Comment 16

[Edouard Oger [:eoger]]

This was part of legacy Sync.