search

fritzing / fritzing-app

Fritzing desktop application
http://fritzing.org
Other
3.98k stars 826 forks source link

lots of redundant calculations in enabling/disabling menu items #263

Open davidperrenoud opened 10 years ago

davidperrenoud commented 10 years ago

From irasc...@gmail.com on November 18, 2008 02:20:02

maybe there should just be a single enableAllMenus function?

Original issue: http://code.google.com/p/fritzing/issues/detail?id=263

davidperrenoud commented 10 years ago

From irasc...@gmail.com on September 28, 2010 12:37:08

Labels: Component-Code

davidperrenoud commented 10 years ago

From irasc...@gmail.com on August 13, 2011 08:12:31

maybe the time to update all menus is when the selection changes?

Labels: -Type-Defect Type-Enhancement

davidperrenoud commented 10 years ago

From irasc...@gmail.com on September 19, 2011 07:19:29

consider a MenuManager class and moving all the MainWindow menu stuff there