General cleanup of Base - #1299
Conversation
Sorry, something went wrong.
I started cleaning up some of `Base`'s startup sequence for clarity of what is being started when. Nowhere near completion and I think a lot of this class will need to be refactored in the future. Also removed some of the timing measurement comments Added some comments to the Processing CLI class
|
|
||
| // // Check if there were previously opened sketches to be restored | ||
| // boolean opened = restoreSketches(); | ||
| ContributionListing cl = ContributionListing.getInstance(); |
There was a problem hiding this comment.
This is one thing that moved in the sequence, I think this should be okay
Sorry, something went wrong.
There was a problem hiding this comment.
what is contribution listing?
Sorry, something went wrong.
There was a problem hiding this comment.
It's the helper class that will gather the contributions in the contributions manager from the net
Sorry, something went wrong.
| private static void setLookAndFeel() { | ||
| try { | ||
| // Use native popups to avoid looking crappy on macOS | ||
| JPopupMenu.setDefaultLightWeightPopupEnabled(false); |
There was a problem hiding this comment.
Also moved in the sequence
Sorry, something went wrong.
catilac
left a comment
There was a problem hiding this comment.
lgtm! merge when ready
Sorry, something went wrong.
I started cleaning up some of Base's startup sequence for clarity of what is being started when. Nowhere near completion and I think a lot of this class will need to be refactored in the future.
Also removed some of the timing measurement comments
Added some comments to the Processing CLI class