Closed Bug 826335 Opened 7 years ago Closed 5 years ago

globo.com sends desktop content to Firefox OS

Categories

(Web Compatibility :: Mobile, defect)

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(b2g-v2.0 affected, b2g-v2.1 affected)

RESOLVED FIXED
Tracking Status
b2g-v2.0 --- affected
b2g-v2.1 --- affected

People

(Reporter: lmandel, Assigned: karlcow)

References

()

Details

(Whiteboard: [uaoverride] [country-br] [sitewait] [clientsniff])

globo.com serves a desktop site to B2G. Bug 819210 added an UA override for this domain. The override should be removed once the domain serves mobile content to B2G. This bug tracks the evangelism for the site to recognize the B2G UA and the removal of the override at that time.
Whiteboard: [uaoverride] → [uaoverride] [country-br]
Trying to check how the redirection is done. This seems to be done through JS and not at the HTTP level. I put the relevant piece of code into gist.

https://gist.github.com/karlcow/5977291#file-gistfile1-js-L5

 'matcherMobile': /android.*mobile|blackberry|docomo|fennec|htc|ip(?:hone|od)|lge|mot|netfront|nokia|opera\sm(?:ini|obi)|samsung-gt-s8500|skyfire|up\.browser|vx10000|wap|windows\s(?:ce|mobile\s7|phone\sos\s7)/i,
'matcherNotMobile': /android.*(gt-(?:p|n)|shw-|sc-01c|sch-i800|f-01d|ct100)|tablet|xoom/i,
'matcherMobileReferrer': /m\.globo\.com/,

Firefox OS User agent string is 

"Mozilla/5.0 (Mobile; rv:18.0) Gecko/18.0 Firefox/18.0" 

It is not matching any of the mobile keyword. I would encourage you to match any mobile. There is a tablet in the NotMobile so that will fix for those who have "Mobile Tablet". Better would be responsive design and features detection, but that is another jam. :)

I will contact people at Globo.
Contacted someone at Globo on 2013-07-11. No answer yet.
Assignee: nobody → kdubost
Whiteboard: [uaoverride] [country-br] → [uaoverride] [country-br] [sitewait]
Hallvord, this is not fixed. False positive for the extension.
Flags: needinfo?(hsteen)
Whiteboard: [uaoverride] [country-br] [sitewait] → [uaoverride] [country-br] [sitewait] [clientsniff]
extension test fixed, thanks
Flags: needinfo?(hsteen)
See Also: → 826845
Trying to have contact again.
Status: NEW → ASSIGNED
Hello, I contacted 2 friends from Globo.com Andre Fonseca and Davidson Fellipe. They are web developers and maybe can help us on this.
Contacted one of my contacts there to see if we could move forward. There is no resolution on this.
sent a reminder yesterday but no answers :/
I have been trying to contact someone else through techtudo issue who is the Mobile Product Manager of Globo.com. So we will see.
Navigating to Globo.com on 2.0 Flame device now takes user to m.globo.com

However, navigating to oglobo.globo.com on same device does not direct user to m.oglobo.globo.com
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
Brogan, did you remove the UA override before testing?
https://hg.mozilla.org/mozilla-central/file/tip/b2g/app/ua-update.json.in

Globo is still not sending Firefox OS to the mobile content.

For oglobo.globo.com, please open a separate bug. 
Thanks for pointing out this one.
You can use that form: https://bugzilla.mozilla.org/form.mobile.compat
Flags: needinfo?(bzumwalt)
Summary: globo.com doesn't recognize B2G UA as mobile → globo.com sends desktop content to Firefox OS
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage+][lead-review+]
ua-update.json was not removed before testing. When globo.com rule was removed the issue was found to still reproduce.

Bug filed for oglobo.globo.com issue: https://bugzilla.mozilla.org/show_bug.cgi?id=1057114
QA Whiteboard: [QAnalyst-Triage+][lead-review+] → [QAnalyst-Triage?][lead-review+]
Flags: needinfo?(bzumwalt) → needinfo?(ktucker)
QA Whiteboard: [QAnalyst-Triage?][lead-review+] → [QAnalyst-Triage+][lead-review+]
Flags: needinfo?(ktucker)
QA Whiteboard: [QAnalyst-Triage+][lead-review+] → [QAnalyst-Triage?][lead-review+]
Flags: needinfo?(ktucker)
QA Whiteboard: [QAnalyst-Triage?][lead-review+] → [QAnalyst-Triage+][lead-review+]
Flags: needinfo?(ktucker)
See Also: → 1057114
Karl, I got a tweet today saying they've launched a fully responsive site now: https://twitter.com/leobalter/status/535867000783118336. Can you test with override disabled?
I've tested this on Firefox OS 1.1 and 1.2 already (using the Bocoup ODL devices), it's working just fine. Touch events for horizontaly sliding content, opening accordion styled content, etc.

I would say it's fixed. 

Btw, it didn't find this issue searching on https://webcompat.com/issues, that's why I've sent the tweet.
Yes Working for me too.
This is fixed! Finally and for anyone. Hurrah!
We can also remove the UA override.
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Flags: needinfo?(kdubost)
Resolution: --- → FIXED
Blocks: 1103680
Blocks: 1171828
No longer blocks: 1171828
Product: Tech Evangelism → Web Compatibility
You need to log in before you can comment on or make changes to this bug.