Regarding - #10167
Does this look good?
@mattab - Really sorry that I mistakenly committed code from 2 different issues. So, this pull request now also contains fix for the issue #10164
Recommended should be translated into the other languages.
@auchri - Ok thanks! I will fix it. Is this why the test failed?
I think the PhantomJs crash is not related to your change. I will restart the test
@tsteur @auchri - I have updated the code. Please let me know if there is anything else which needs to be changed :)
The test is still failing because of pixel difference. Can it be caused because of the fact that I have added the (Recommended) word?
You can see the UI changes / the failing tests here: http://builds-artifacts.piwik.org/piwik/piwik/master/19281/ and they look good. We will fix them once this PR is merged
Can someone confirm the PR works as expected? I presume we might need to test this in different browsers.
Thanks @blueelvis for the PR. did you maybe test the iframe code in several browsers such as IE and Firefox? according to https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/X-Frame-Options it won't work on Chrome anyway..
Please close this PR because for some unknown reason, this is not getting updated. New PR has been created #10317