openmc-dev / plotter

Native plotting GUI for model design and verification
MIT License
46 stars 18 forks source link

fix duplicate calls to makePlot() on startup #84

Closed kkiesling closed 2 years ago

kkiesling commented 2 years ago

This addresses #67. See discussion there for why these lines were causing multiple loading of the plot.

Removal of these 3 lines means that the state of these three toggled options in the Edit menu are not saved for the next time the window is opened. If we want the behavior to be such that the state is saved, then I don't think we can get around this duplicate loading issue (which can happen up to 3 times, one for each toggled option). If people are okay with that state not being saved, then it should be as simple as just removing these three lines to fix it.