search

Esri / solutions-widgets-wpf

Solutions Widgets for use in the ArcGIS WPF Operations Dashboard.
Apache License 2.0
15 stars 15 forks source link

OrderOfBattle - very minor - just remove or explain commented out code #14

Closed csmoore closed 10 years ago

csmoore commented 10 years ago

Just go through the commented out code in the project and either remove or add some comments to explain why you are keeping

https://github.com/ArcGIS/solutions-widgets-wpf/tree/master/source/OrderOfBattle/OrderOfBattle

Example:

https://github.com/ArcGIS/solutions-widgets-wpf/blob/master/source/OrderOfBattle/OrderOfBattle/OOBCache.cs#L59

pHill5136 commented 10 years ago

Removed code comments with issue #17 merge

csmoore commented 10 years ago

Looks like just 1 file was missed so keeping open as a reminder:

https://github.com/Esri/solutions-widgets-wpf/blob/master/source/OrderOfBattle/OrderOfBattle/Config/OOBWidgetDialog.xaml.cs#L24

https://github.com/Esri/solutions-widgets-wpf/blob/master/source/OrderOfBattle/OrderOfBattle/Config/OOBWidgetDialog.xaml.cs#L410

csmoore commented 10 years ago

I did this when reviewing/resolving #19