Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HDX: load wpt files multiple times bug #117

Closed
jteresco opened this issue Jul 14, 2017 · 3 comments
Closed

HDX: load wpt files multiple times bug #117

jteresco opened this issue Jul 14, 2017 · 3 comments
Assignees

Comments

@jteresco
Copy link
Contributor

If you load a WPT file, then load another, the route is OK, but the "raw file contents" area then has the contents of both files.

@jteresco jteresco added the bug label Jul 14, 2017
@jteresco jteresco self-assigned this Jul 14, 2017
@jteresco
Copy link
Contributor Author

Need to be considering this as we test and debug @mdagsjr's UI redesign.

@yakra
Copy link
Collaborator

yakra commented Jul 16, 2018

I haven't looked under the hood here, but this looks like a case of a Javascript foo.innerHTML += "yadda yadda" that should instead be foo.innerHTML = "yadda yadda"

@jteresco
Copy link
Contributor Author

jteresco commented Aug 4, 2018

This was a little insidious, and affected all file types. It's essentially what @yakra suggested, but appending the entire table to the contents of the div for the loaded data (the "contents_table"). Fix forthcoming for the hdxt version, will get into the production version later this month.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants