spolu / breach_core

A Browser written in JS. Free. Modular. Hackable.
https://breach.github.io/breach_core/
MIT License
5.41k stars 412 forks source link

Calling tabs_close will not actually close the tab #189

Closed nearsyh closed 10 years ago

nearsyh commented 10 years ago

After calling the tabs_close, the page will be blank, but the tab won't disappear.

I think the reason is that to actually delete the tab displayed by mod_strip, we need to do this in Line361. However calling tabs_close which is in the breach_core module won't do this. Adding a tabs:close event may be good?

spolu commented 10 years ago

There my.core_tabs is local to mod_strip and has nothing to do with any object in breach_core... right?

nearsyh commented 10 years ago

Exactly. But the problem is when I call tabs_close in other modules, the corresponding tab displayed by mod_strip is not closed (only become blank).

spolu commented 10 years ago

Yep... We should probably add some code in mod_strip to handle that case. Would you feel like making a PR on mod_strip?

nearsyh commented 10 years ago

Sure

spolu commented 10 years ago

Awesome! :)