Last modified: 2012-02-24 02:52:29 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T33702, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 31702 - Android app shows blank screen after network failure
Android app shows blank screen after network failure
Status: RESOLVED FIXED
Product: Wikipedia App
Classification: Unclassified
Generic (Other open bugs)
1.0.0 (Android)
All All
: Highest critical
: ---
Assigned To: Anis Kadri
:
Depends on: 31447
Blocks:
  Show dependency treegraph
 
Reported: 2011-10-14 18:37 UTC by Tomasz Finc
Modified: 2012-02-24 02:52 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: Android
Assignee Huggle Beta Tester: ---


Attachments

Description Tomasz Finc 2011-10-14 18:37:05 UTC
If we fail to retrieve data from a network the view just shows a blank page. You can access buttons but can't reload the page in any way.
Comment 1 Brion Vibber 2011-10-21 18:28:51 UTC
Anis, can you include a link to the commit that resolves this please? Thanks!
Comment 3 Brion Vibber 2011-10-21 18:41:41 UTC
That commit has the comment "fixing nearme returned urls"; is that also a general fix for the above-noted network errors?
Comment 4 Anis Kadri 2011-10-21 18:45:14 UTC
Yes it should be. Trying disabling your network connection and launch the app.
Comment 5 Brion Vibber 2011-10-21 18:47:18 UTC
The reason I ask is that that commit only changes the NearMeActivity and WikiItemizedOverlay Android plugin pieces, and tweaks the width of a textview. As far as I know none of these activities are in use when we start the app?
Comment 6 Anis Kadri 2011-10-21 19:36:36 UTC
It is actually an earlier commit

https://github.com/nitobi/Wikipedia/commit/eea4be11efc3d22c3981679becb29ccc03d6d665

but it works with the latest revision of master

I basically added a timeout and an error handler
Comment 7 Brion Vibber 2011-10-21 20:20:34 UTC
Thanks. Please always point to the actual revisions which actually make changes, otherwise we can't tell what you're talking about and it makes review and maintenance a lot harder.

What's with replacing jQuery with this Zepto thing? Why is that done together with a stop button? Is that a requirement? Are there any other benefits? Please be sure to document such things when making changes!
Comment 8 Anis Kadri 2011-10-21 20:33:14 UTC
Sorry about that...

Zepto is a much lighter version of jQuery optimized for mobile. jQuery has a lot of desktop browser specific code (IE 6 specific code for instance) that we don't need for mobile. The syntax is the same. It is missing some jQuery functions but the main ones are supported. It also includes some other functions only available on webkit mobile.

https://github.com/madrobby/zepto

The stop button change is totally unrelated to Zepto. I just happened to make both changes at the same time. 

I will from now on submit the commit url every time I update a ticket to RESOLVE. Apologies...

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links