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

Reader: Escape background image urls in inline CSS #9428

Merged
merged 1 commit into from
Nov 16, 2016
Merged

Conversation

blowery
Copy link
Contributor

@blowery blowery commented Nov 16, 2016

Fixes #9413

See 9413 for repro and test

@matticbot
Copy link
Contributor

@blowery blowery added [Feature] Reader The reader site on Calypso. [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. labels Nov 16, 2016
@blowery blowery added this to the Reader Refresh: Cards Cleanup milestone Nov 16, 2016
@blowery
Copy link
Contributor Author

blowery commented Nov 16, 2016

cc @jancavan @alisterscott

@jancavan
Copy link
Contributor

I'm seeing the thumbnail now 🚢

@jancavan jancavan added [Status] Ready to Merge and removed [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. labels Nov 16, 2016
@blowery blowery merged commit 2f78f07 into master Nov 16, 2016
@blowery blowery deleted the fix/reader/9413 branch November 16, 2016 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Reader The reader site on Calypso.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants