joewalker / devtools.html

19 stars 7 forks source link

update /devtools/client/ -> /client/ #14

Open bgrins opened 8 years ago

bgrins commented 8 years ago

@jlongster please have a look at this plus the updated doc at https://github.com/joewalker/devtools.html#running-the-server-and-using-the-toolbox. I believe this will fix all of our resource path issues via gulp start.

bgrins commented 8 years ago

This also fixes the debugger icon

jlongster commented 8 years ago

Looks generally good to me, r+ although I you and jsantell are talking on IRC about this. Haven't thought too much about the implications, so we just need to decide one way or the other.

bgrins commented 8 years ago

I'm going to hold off on this change since things are pretty much working as-is and I don't want to break things. There's something funny with trying to load the inspector on itself now, but I'm not sure when that stopped working (reverting https://github.com/joewalker/devtools.html/commit/6d79aeefbd2cc76423580cfa58b755cac2ffd13c doesn't seem to help).