Change:
id: 587d78b1367417b2b2512b09 title: Make an Image Responsive challengeType: 0 <del> videoUrl: 'https://scrimba.com/p/pzrPu4/cz763UD' </del> forumTopicId: 301140 dashedName: make-an-image-responsive ---

Discussion:
<!-- NOTE: If you're reporting a security issue, don't create a GitHub issue. Instead, email We will look into it immediately. --Describe your problem and how to reproduce it: I followed along with the video entered the code exactly and the first test fails second test passes the code is correct so the test I think is broke Add a Link to the page with the problem: Tell us about your browser and operating system: Browser Name: microsoft edge Browser Version: Operating System: windows 10 If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):
worked in chrome only test I have had trouble with so far
Please use for debugging the code of your projects. If the forum members determine there is nothing wrong with your code, you can request this issue to be reopened. GitHub issues are meant for reporting bugs and not troubleshooting the code in your projects. Thank you for understanding.
not n issue with code issue with test it worked in chrome and not edge test is broken on Microsoft edge
What code did you try on Microsoft Edge? What errors (if any) did you see in the browser's console (not the FCC console)?
I followed the code in the video for help
Please post what you see in your browser's console. Thank you. Also, please post the code you tried.
I notice if I add the following to the first test's testString (when testing locally): I only see the following in Edge vs the following in Chrome:
What version of Edge are you running? and It passes for me in the Chromium-based version which is the only version I have, so I'd like to know what version you are running. Does it give you the same output if you run the log from the browser console? I get the full object when I run it in the console. Not sure where it is getting from, I only see that on the fa icons (animated icons) and in the bootstrap file for the main site. I don't think it should show up in the output/test iframe context. I don't see it in the object when I run the log in the browser console. Side note, not really sure why the test is using getComputedStyle, as far as I know using should be exactly the same. It's a bit shorter and maybe less confusing than mixing Jquery and plain JS.

Or just switching it to only plain JS for all of it.
I tested it using Edge version 44..449.0 on Windows 10 Home. It gives the same from the browser console. On Chrome it works as expected. FYI - When I in the browser console, I see not using Edge. For Chrome, I see .
Well, I can't test it using that version (unless I set up a VM for it). Not sure where it is getting the pixel value from, that seems very odd. I don't understand how set to 100% would ever compute to a pixel value. Using should be the same as using getComputedStyle. Looking at the jQuery source I'm not sure why it wouldn't be the same. Jquery source for reference
Can I take a look at this issue?
sure, go for it.
For these challenges, are the helper-functions ready to be used in the tests? Or, can a simple fix be made like:
That is what it was before So I don't see why not. As for the helper-functions, I don't know anything about that. I'd still like to know why Edge is giving the result it is, but I guess it doesn't matter much.
Well, , you were involved with the current change. Are we better leaving it as is, changing the test to use regex, or change it to what it was before ?
No, the helper functions aren't ready Looks like the method won't work for edge. So yes, probably need to go back to regex here if we want it to work on edge. Here's a modification of something I have used in the past to isolate the styles of a selector... ^ I modified without testing, hoping to get a helper function for isolating styles to the PR for those.
This is still causing problems every now and then. Have we abandoned the support of the older Edge versions (prior to the switch to Chromium), or should we fix this by switching back to a regex?