Skip to content

Fixed 404 fail resource in plugins page #844

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

Merged
merged 1 commit into from
Feb 9, 2017
Merged

Conversation

garmjs
Copy link

@garmjs garmjs commented Feb 7, 2017

Removed section variable from renderer.image in file markdown.js because it will fail to load resource ( jpg, gif, svg, mp4 )

url with failed resources can be found here.
https://webpack.js.org/plugins/npm-install-webpack-plugin
https://webpack.js.org/plugins/uglifyjs-webpack-plugin/

Actual Behavior in pages

screen shot 2017-02-07 at 4 24 09 pm

screen shot 2017-02-07 at 4 24 38 pm

screen shot 2017-02-07 at 5 13 01 pm

Removed section variable from renderer.image in markdown.js because it will fail to load resource ( img, svg, mp4 )

https://webpack.js.org/plugins/npm-install-webpack-plugin
https://webpack.js.org/plugins/uglifyjs-webpack-plugin/
@garmjs
Copy link
Author

garmjs commented Feb 7, 2017

@simon04 review

@simon04
Copy link
Collaborator

simon04 commented Feb 9, 2017

@bebraw can we drop the section part of the href of images?

If yes: the line section = … plus comment should be removed as well.

@simon04 simon04 requested a review from bebraw February 9, 2017 11:15
@bebraw
Copy link
Contributor

bebraw commented Feb 9, 2017

@simon04 Yeah, most likely. At least the link integrity check passes so it should be fine. I would still check locally to be sure, though.

@garmjs
Copy link
Author

garmjs commented Feb 9, 2017

@bebraw checked localy everything looks good without section

@bebraw bebraw merged commit d5677a1 into webpack:master Feb 9, 2017
@bebraw
Copy link
Contributor

bebraw commented Feb 9, 2017

Ok. 👍

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

Successfully merging this pull request may close these issues.

3 participants