mozilla / brackets

An open source code editor for the web, written in JavaScript, HTML and CSS.
http://brackets.io
MIT License
174 stars 276 forks source link

Images won't load if they have focus when Bramble starts #494

Open humphd opened 8 years ago

humphd commented 8 years ago

STR:

screenshot 2015-11-02 15 13 08

humphd commented 8 years ago

This is only an issue in Bramble vs. Thimble, it seems, because in Thimble we always try to load an HTML file.

humphd commented 7 years ago

Please test my PR in https://github.com/mozilla/brackets/pull/625 with this bug, I might have fixed it when I rewrote that code.

peiying16 commented 7 years ago

I want to test it. I use following commands to download your branch, where are the codes it downloads to? This is the first time for me to test others' code.

git remote add "humphd" https://github.com/humphd/brackets.git
git fetch humphd
humphd commented 7 years ago
git remote add humphd https://github.com/humphd/brackets.git
git fetch humphd
git checkout livedev-fixes
peiying16 commented 7 years ago

I checkout to your branch, but I can't load the page. Error message is here. err

peiying16 commented 7 years ago

Here are my branchs and remote git

humphd commented 7 years ago

I would clear your cache. Let's discuss in class tomorrow too.

humphd commented 7 years ago

@peiying16 thanks for altering me to this--it was a bug in my code. I've pushed a new commit on my PR which fixes it. Do you mind trying again? Thanks for letting me know about this, I totally missed it!

peiying16 commented 7 years ago

@humphd Hello, I try your PR. Now I can load the page successfully, but it doesn't fix this issue. It is still 404 image

humphd commented 7 years ago

OK, this is helpful. Thank you. I think it's something to do with it trying to load an ImageView vs. an HTML Live Document on startup, and not using a rewritten Blob URL. We'll have to debug this separate to what you're doing.

Thanks for you testing, I just hoped it would solve this bug, too.

humphd commented 7 years ago

NOTE: I have a fix for this in my work for #725, see https://github.com/mozilla/brackets/pull/725/commits/a122b49a16170be920610dd288273d2858cccfb6