Unidata / IDV

The Integrated Data Viewer (IDV) from Unidata is a framework for analyzing and displaying geoscience data.
http://www.unidata.ucar.edu/software/idv/
Other
80 stars 37 forks source link

Free drawing and Front drawing have separate tables and column labels [1536] #65

Closed tommyjasmin closed 10 years ago

tommyjasmin commented 10 years ago

Hi guys -

This fixes a problem where Free Drawing control and Front Drawing controls where sharing the same JTable class (an extension of AbstractTableModel). As a result the tables were mis-labeled. We just gave the Front control its own class, which returns the proper column label. Me, Jon, and Mike looked at this together all all agreed it's a decent, clean solution.

Both pics below show corrected behavior, you can compare with current behavior.

Free drawing: Displays -> Special -> Drawing Control

Front drawing: Data Choosers -> Fronts -> Add Source

screen shot 2014-03-06 at 3 35 02 pm

screen shot 2014-03-06 at 3 34 24 pm

tommyjasmin commented 10 years ago

One more thing guys - after accept, you will want to throw the standard Unidata copyright at the top of FrontDrawingControl.java

julienchastang commented 10 years ago

Thanks @tommyjasmin. Merged.