Closed
Bug 1006775
Opened 11 years ago
Closed 11 years ago
Web Literacy competency view does not render in Internet Explorer
Categories
(Webmaker Graveyard :: webmaker.org, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: akirose, Assigned: jon)
References
Details
(Whiteboard: [resources][2014Q2][June13])
Attachments
(2 files, 1 obsolete file)
Steps to reproduce
1. load webmaker.org/explore (v1.0.83) on staging in IE 10
2. Click on any competency in left nav
Console result:
`InvalidStateError<div class="explore-main ng-scope" ng-view="">`
Visual result:
Navigation item becomes highlighted, view template loads, but never fully renders
(browser screenshot attached)
Updated•11 years ago
|
Whiteboard: [explore][r3] → [textbook][soft-launch]
Updated•11 years ago
|
Whiteboard: [textbook][soft-launch] → [resources][2014Q2][May30]
Updated•11 years ago
|
Assignee: nobody → aki
Status: NEW → ASSIGNED
Reporter | ||
Updated•11 years ago
|
Summary: /explore competency view does not render properly in IE 10 → Web Literacy competency view does not render properly in IE 10
Reporter | ||
Comment 2•11 years ago
|
||
full console output:
HTML1300: Navigation occurred.
File: weblit-Remix
InvalidStateError <div class="explore-main ng-scope" ng-view="">
{
[functions]: ,
__proto__: { },
ABORT_ERR: 20,
code: 11,
constructor: { },
DATA_CLONE_ERR: 25,
DOMSTRING_SIZE_ERR: 2,
HIERARCHY_REQUEST_ERR: 3,
INDEX_SIZE_ERR: 1,
INUSE_ATTRIBUTE_ERR: 10,
INVALID_ACCESS_ERR: 15,
INVALID_CHARACTER_ERR: 5,
INVALID_MODIFICATION_ERR: 13,
INVALID_NODE_TYPE_ERR: 24,
INVALID_STATE_ERR: 11,
message: "InvalidStateError",
name: "InvalidStateError",
NAMESPACE_ERR: 14,
NETWORK_ERR: 19,
NO_DATA_ALLOWED_ERR: 6,
NO_MODIFICATION_ALLOWED_ERR: 7,
NOT_FOUND_ERR: 8,
NOT_SUPPORTED_ERR: 9,
PARSE_ERR: 81,
QUOTA_EXCEEDED_ERR: 22,
SECURITY_ERR: 18,
SERIALIZE_ERR: 82,
SYNTAX_ERR: 12,
TIMEOUT_ERR: 23,
TYPE_MISMATCH_ERR: 17,
URL_MISMATCH_ERR: 21,
VALIDATION_ERR: 16,
WRONG_DOCUMENT_ERR: 4
}
"<div class="explore-main ng-scope" ng-view="">"
Reporter | ||
Updated•11 years ago
|
Summary: Web Literacy competency view does not render properly in IE 10 → Web Literacy competency view does not render in Internet Explorer
Reporter | ||
Comment 3•11 years ago
|
||
Updated screen shot
Attachment #8418277 -
Attachment is obsolete: true
Reporter | ||
Updated•11 years ago
|
Assignee: aki → nobody
Reporter | ||
Updated•11 years ago
|
Status: ASSIGNED → NEW
Comment 4•11 years ago
|
||
I think I found the problem! Applying this simple fix to the MakeAPI client library makes things work on my end:
https://github.com/mozilla/makeapi-client/pull/38
On a somewhat more worrisome note, though: the original exception thrown by IE is being masked in the webmaker.org code, which is why it was so hard to discover--I ended up having to just drop console.log()'s everywhere because IE11's debugger wasn't even treating it as an exception. On the smoke test I wrote for makeapi-client, though (see https://github.com/mozilla/makeapi-client/issues/37), IE11's error feedback is perfect: the debugger kicks in on the exact line that's causing the crash.
So it seems like that useful information might be caught by the Angular competencyController in public/js/angular/controllers.js, and munged in a way that makes the error very hard to debug in IE11. :(
Assignee | ||
Comment 5•11 years ago
|
||
Attachment #8431068 -
Flags: review?(cade)
Updated•11 years ago
|
Attachment #8431068 -
Flags: review?(cade) → review+
Comment 6•11 years ago
|
||
Is this done, can we close?
Flags: needinfo?(jon)
Whiteboard: [resources][2014Q2][May30] → [resources][2014Q2][June13]
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → jon
Status: NEW → ASSIGNED
Flags: needinfo?(jon)
Comment 7•11 years ago
|
||
Commit pushed to master at https://github.com/mozilla/webmaker.org
https://github.com/mozilla/webmaker.org/commit/4a7775c9ac31b894b464b426252b93c4d9f91191
Fix bug 1006775 - Update to makeapi-client v0.5.25
Updated•11 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 8•11 years ago
|
||
Reporter | ||
Comment 9•11 years ago
|
||
Okay so emoji breaks bugzilla. Atul, you are my hero.
Comment 10•11 years ago
|
||
No problem! Let me know if there are any other IE problems that you need resolved in a pinch. For some reason I find fixing IE issues therapeutic.
You need to log in
before you can comment on or make changes to this bug.
Description
•