From: Michael A. <out...@gm...> - 2014-02-03 17:17:50
|
Subclassing those was what I had originally been doing - I was just wondering if those should be put in the common code area because we were already making the common code aware of that type of round by changing GuiDef. I will continue down the subclassing path. Thanks, Mike On Mon, Feb 3, 2014 at 11:54 AM, Stefan Frey <ste...@we...> wrote: > Mike: > I do not fully understand your proposal. > However the current mechanism would imply to subclass GameManager and > GameUIManager and make adjustments in those 1862 specific subclasses. > > Only the GuiDef.Panel would have an additional entry. This is not > perfect, but as this will require some rework anyway, that is not an > issue. Maybe place a remark to the enum entry that this is for 1862 only. > > Stefan > > > On 02/03/2014 02:39 PM, Michael Alexander wrote: > > By extension, should I then make the common GameManager and > > GameUIManager aware of Parliament rounds and drop the "Round" class for > > it in the common area as well? > > > > Mike > > > > > > On Mon, Feb 3, 2014 at 3:21 AM, Stefan Frey <ste...@we... > > <mailto:ste...@we...>> wrote: > > > > I prefer explicit solutions, so feel free to add another item in the > > GuiDef.Panel. > > > > It would be great to add more generic functionality for window > creation > > and management, but this has not been my scope so far. So feel free > to > > come up with suggestions or otherwise simply follow the existing > > mechanisms. > > > > Stefan > > > > > > On 02/02/2014 11:43 PM, Michael Alexander wrote: > > > I'm making a new window for the Parliamentary Rounds in 1862. > > Ideally, > > > I'd like to be able to extend GuiDef.Panel for the new window, > > however, > > > I wanted to float that with you guys first. > > > > > > Alternately, I could re-use "START_ROUND" - but since it's not > > really a > > > start round, I was hesitant to do that. > > > > > > Which approach would be preferable? > > > > > > Mike > > > > > > > > > > > > ------------------------------------------------------------------------------ > > > WatchGuard Dimension instantly turns raw network data into > actionable > > > security intelligence. It gives you real-time visual feedback on > key > > > security issues and trends. Skip the complicated setup - simply > > import > > > a virtual appliance and go from zero to informed in seconds. > > > > > > http://pubads.g.doubleclick.net/gampad/clk?id=123612991&iu=/4140/ostg.clktrk > > > > > > > > > > > > _______________________________________________ > > > Rails-devel mailing list > > > Rai...@li... > > <mailto:Rai...@li...> > > > https://lists.sourceforge.net/lists/listinfo/rails-devel > > > > > > > > > > ------------------------------------------------------------------------------ > > Managing the Performance of Cloud-Based Applications > > Take advantage of what the Cloud has to offer - Avoid Common > Pitfalls. > > Read the Whitepaper. > > > http://pubads.g.doubleclick.net/gampad/clk?id=121051231&iu=/4140/ostg.clktrk > > _______________________________________________ > > Rails-devel mailing list > > Rai...@li... > > <mailto:Rai...@li...> > > https://lists.sourceforge.net/lists/listinfo/rails-devel > > > > > > > > > > > ------------------------------------------------------------------------------ > > Managing the Performance of Cloud-Based Applications > > Take advantage of what the Cloud has to offer - Avoid Common Pitfalls. > > Read the Whitepaper. > > > http://pubads.g.doubleclick.net/gampad/clk?id=121051231&iu=/4140/ostg.clktrk > > > > > > > > _______________________________________________ > > Rails-devel mailing list > > Rai...@li... > > https://lists.sourceforge.net/lists/listinfo/rails-devel > > > > > > ------------------------------------------------------------------------------ > Managing the Performance of Cloud-Based Applications > Take advantage of what the Cloud has to offer - Avoid Common Pitfalls. > Read the Whitepaper. > > http://pubads.g.doubleclick.net/gampad/clk?id=121051231&iu=/4140/ostg.clktrk > _______________________________________________ > Rails-devel mailing list > Rai...@li... > https://lists.sourceforge.net/lists/listinfo/rails-devel > |