On 12-May-2002 Marc Wilson wrote: > On Sun, May 12, 2002 at 09:30:32AM -0700, Sean 'Shaleh' Perry wrote: >> I am *NOT* talking about removing the workspace from the Workspaces menu. >> We >> are planning on removing the current workspace from the send to menu in the >> Window menu on the window itself. > > What's the point of that? >
the menu says "send to" it makes no sense to send a window to the workspace it is on. The code actually does a noop if you try. The only reason it is there now is because it has to be, there is no other obvious, efficient, clean way to code it. The code does "what is the index of the item in the menu you clicked? ok getWorkspace(index)". So if we were to try to show only say 1,2,4 out of [1,2,3,4] clicking on 4 would send you to 3. In fact all of the menu code in blackbox works by looking at the index of the menu item in the menu and keying off of that. So we have ugly things like loops where we give each window a unique number based on their location in the menu.
