W5: adding a shortcut does not actually make that shortcut appear in the shortcut bar (due to parent page not being reloaded or at least shortcut bar not being reloaded via Ajax/AHAH)
W6: When disabling Toolbar module from within the Overlay, it is still there (again, see W5). This could extend to whatever is displayed outside the overlay (admin_menu, the upper part of the toolbar, any lower bar like l10n_client, etc).
Essentially, the task here is to figure out a way to reload relevant parts of the parent window, and figure out a way to trigger that reload. I'm assuming it's easier to reload just the parts of the page that include the .overlay-displace-top and .overlay-displace-bottom classes, rather than mess with the entire parent window and risk losing the overlay iframe, but whichever way works is great.
Comments
Comment #1
David_Rothstein commentedThis was already taken care of in the initial commit - at least the basic implementation. We might be able to do more magic at some point, but that will be a separate issue.