Universite-Gustave-Eiffel / I-Simpa

An Open Source software for 3D sound propagation modelling
https://i-simpa.univ-gustave-eiffel.fr/
GNU General Public License v3.0
227 stars 56 forks source link

Interface layout save after reinitialization #274

Open wbinek opened 3 years ago

wbinek commented 3 years ago

Describe the bug The save of interface layout is disabled after clicking on "Window->Reinitialize interface" in the session. I am not sure this is a good behaviour. If I mess my layout I use the reinitialization option and then the fixed layout is lost on next app start.

To Reproduce Steps to reproduce the behaviour:

  1. Set wrong/messed layout, close app (the messed layout gets saved)
  2. Start I-SIMPA againt, use Window->Reinitialize to restore starting layout
  3. Close app (the fixed layout is not saved)
  4. Start app again (messed layout is restored)

Expected behaviour I believe that it would be better to save app layout regardless of reinitialization.

Desktop (please complete the following information)

Additional context The behaviour can be changed by commenting/removing the 645 line in i_simpa_main.cpp file: saveManagerConfig=false I can create the pull request for this issue but I wanted to discuss the issue first.