From: Stefan F. <ste...@we...> - 2012-01-12 18:12:22
|
Erik: To answer your questions: 1. would only de-activate those menu-items which are corresponds to Rails actions. All others should be still active. 2. message pop-ups should never be modal (or quasi-modal, as you are currently implementing), most likely they could be replaced by displaying it in the info/status panel (adding an info panel in the stock window would allow that). Some other remarks: * Most likely it makes sense to merge your commit with the code that deactivates the windows when the game history is used, which tries to achieve a similar effect. * More general: I think there in principle there is no general difference between the non- model popup requests and any other situation: Only those buttons and menu- items should be active that are either available generally (like save/load, zoom, show/hide windows) or are allowed actions at that time. Currently there are some other cases when buttons are still active, which do not contain active commands, that generate error messages if pressed. The ideal solution would be if the UI would be able to figure out what to activate and deactivate given the current set of actions and maybe a few flags. I know that the current UI already tries to achieve that already, but there are some bits still missing. Stefan On Thursday, January 12, 2012 04:39:58 pm Erik Vos wrote: > I have pushed a change to disable all buttons (in the current window) as > long as a non-modal popup request has not been answered. > Two questions remain, on which I'm in doubt: > 1. Should (most) menu items also be disabled? > 2. Should we make an exception to message-only popups? I currently do. > > Erik. > > > --------------------------------------------------------------------------- > --- RSA(R) Conference 2012 > Mar 27 - Feb 2 > Save $400 by Jan. 27 > Register now! > http://p.sf.net/sfu/rsa-sfdev2dev2 > _______________________________________________ > Rails-devel mailing list > Rai...@li... > https://lists.sourceforge.net/lists/listinfo/rails-devel |