Skip to content

Closes #1020 - Move Comment field from Comment tab to QSO tab#1033

Open
ea5wa wants to merge 1 commit into
ea4k:masterfrom
ea5wa:issue-1020-move-comment-to-qso
Open

Closes #1020 - Move Comment field from Comment tab to QSO tab#1033
ea5wa wants to merge 1 commit into
ea4k:masterfrom
ea5wa:issue-1020-move-comment-to-qso

Conversation

@ea5wa

@ea5wa ea5wa commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Closes #1020

Summary

  • Moves the Comment field from its own dedicated tab (MainWindowInputComment) into the QSO tab (MainWindowInputQSO), saving users a tab click on every QSO
  • The field is placed on a full-width row below Name/Power, with the Keep checkbox at the right end and consistent spacing with the rest of the tab
  • Removes MainWindowInputComment class entirely; all callers in MainWindow migrated to QSOTabWidget

Behaviour preserved

  • getComment() / setComment() — moved to MainWindowInputQSO
  • Keep this data for next QSO checkbox — keepCommentCheckBox added next to the field
  • clear() respects the keep flag
  • getQSOData() / setQSOData() include the comment field
  • Backup/restore QSO (backupCurrentQSO / restoreCurrentQSO) updated accordingly

Files affected

  • src/inputwidgets/mainwindowinputqso.h/.cpp — new field, methods, layout
  • src/inputwidgets/mainwindowinputcomment.h/.cpp — deleted
  • src/mainwindow.h/.cpp — member removed, all calls migrated
  • src/CMakeLists.txt — source removed
  • tests/tst_mainwindow/CMakeLists.txt / tst_mainwindow.cpp — references removed

Test plan

  • Comment field visible in QSO tab below Name/Power row
  • Comment saved and loaded correctly when editing an existing QSO
  • Keep checkbox retains comment across QSOs when checked
  • Comment cleared when Keep is unchecked and a new QSO starts
  • No Comment tab present anymore

🤖 Generated with Claude Code

Add commentLineEdit and keepCommentCheckBox to MainWindowInputQSO,
placed on a full-width row below Name/Power with equal spacing.
Remove MainWindowInputComment class entirely and migrate all callers
in MainWindow to use QSOTabWidget instead.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ea5wa

ea5wa commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

Build and Checked in windows. Working perfectly

@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move Comment field from its own tab into the QSO tab

1 participant