IJHack / QtPass / 23825672923

20%

master: 7%

push

github

fix: resolve remaining TODOs and save window state (#877)

* fix: resolve remaining TODOs and save window state

- Remove TODO from on_toolButtonStore_clicked (comment was outdated)
- Implement window size/position saving in ConfigDialog::closeEvent
- Implement window size/position saving in KeygenDialog::closeEvent
- Implement window size/position saving in UsersDialog::closeEvent
- Convert TODO comment in qtpass.cpp to a note (existing code is correct)

- Add dialog-specific settings to avoid clobbering mainwindow state:
  * Add QtPassSettings::getDialogGeometry/setDialogGeometry
  * Add QtPassSettings::getDialogPos/setDialogPos
  * Add QtPassSettings::getDialogSize/setDialogSize
  * Add QtPassSettings::isDialogMaximized/setDialogMaximized

- Update ConfigDialog, KeygenDialog, UsersDialog to use dialog-specific
  settings with unique keys (e.g., 'configDialog', 'keygenDialog', 'usersDialog')
- Add SettingsConstants for dialog-specific settings

* docs: add doxygen documentation and unit tests for dialog settings

29 of 59 new or added lines in 5 files covered. (49.15%)

1023 of 5097 relevant lines covered (20.07%)

7.88 hits per line