Skip to content

Message Inbox Enhancements#181

Merged
Chris-AC9KH merged 1 commit into
JS8Call-improved:masterfrom
rruchte:enhancement_dockable-msg-window
Feb 6, 2026
Merged

Message Inbox Enhancements#181
Chris-AC9KH merged 1 commit into
JS8Call-improved:masterfrom
rruchte:enhancement_dockable-msg-window

Conversation

@rruchte

@rruchte rruchte commented Feb 5, 2026

Copy link
Copy Markdown
Collaborator

Allow message inbox to dock into the main UI
Enable column sorting in the message table
Rework mark-as-read signals to trigger on row deselection Add SemiSortableHeader subclass of QHeaderView

Allow message inbox to dock into the main UI
Enable column sorting in the message table
Rework mark-as-read signals to trigger on row deselection
Add SemiSortableHeader subclass of QHeaderView
@rruchte rruchte added enhancement New feature or request UI labels Feb 5, 2026
@rruchte rruchte added this to the 2.6.0 milestone Feb 5, 2026
@Chris-AC9KH

Copy link
Copy Markdown
Collaborator

Previously tested, merged! Thanks Rob!

@Chris-AC9KH Chris-AC9KH merged commit e56d3af into JS8Call-improved:master Feb 6, 2026
@Joe-K0OG

Joe-K0OG commented Feb 6, 2026

Copy link
Copy Markdown
Collaborator

For what it's worth - maybe nothing...

Build warnings on Linux Mint 22.3

[ 96%] Building CXX object CMakeFiles/JS8Call.dir/JS8_UI/WideGraph.cpp.o
/js8call/js8call-prefix/src/JS8_UI/MessagePanel.cpp: In member function ‘void MessagePanel::populateMessages(QList<std::pair<int, Message> >)’:
/js8call/js8call-prefix/src/JS8_UI/MessagePanel.cpp:130:21: warning: ‘void QDateTime::setTimeSpec(Qt::TimeSpec)’ is deprecated: Use setTimeZone() instead [-Wdeprecated-declarations]
  130 |       ts.setTimeSpec(Qt::UTC);
      |       ~~~~~~~~~~~~~~^~~~~~~~~
In file included from /Qt/6.9.3/gcc_64/include/QtCore/qcborvalue.h:11,
                 from /Qt/6.9.3/gcc_64/include/QtCore/qjsonvalue.h:8,
                 from /Qt/6.9.3/gcc_64/include/QtCore/qjsonobject.h:8,
                 from /Qt/6.9.3/gcc_64/include/QtCore/QJsonObject:1,
                 from /js8call/js8call-prefix/src/JS8_Main/Message.h:10,
                 from /js8call/js8call-prefix/src/JS8_Main/Inbox.h:8,
                 from /js8call/js8call-prefix/src/JS8_UI/MessagePanel.h:3,
                 from /js8call/js8call-prefix/src/JS8_UI/MessagePanel.cpp:1:
/Qt/6.9.3/gcc_64/include/QtCore/qdatetime.h:407:10: note: declared here
  407 |     void setTimeSpec(Qt::TimeSpec spec);
      |          ^~~~~~~~~~~
[ 97%] Building CXX object CMakeFiles/JS8Call.dir/JS8_Widgets/CheckableItemComboBox.cpp.o
[ 98%] Building CXX object CMakeFiles/JS8Call.dir/JS8_Widgets/LazyFillComboBox.cpp.o
[ 99%] Building C object CMakeFiles/JS8Call.dir/vendor/sqlite3/sqlite3.c.o
[100%] Linking CXX executable JS8Call
[100%] Built target JS8Call

Build warnings on Windows JTSDK64-Tools build system

[ 95%] Building CXX object CMakeFiles/JS8Call.dir/JS8_UI/MessageReplyDialog.cpp.obj
[ 96%] Building CXX object CMakeFiles/JS8Call.dir/JS8_UI/MessagePanel.cpp.obj
C:\JTSDK64-Tools\tmp\wsjtx\JS8_UI\MessagePanel.cpp: In member function 'void MessagePanel::populateMessages(QList<std::pair<int, Message> >)':
C:\JTSDK64-Tools\tmp\wsjtx\JS8_UI\MessagePanel.cpp:130:21: [ 97%] Building CXX object CMakeFiles/JS8Call.dir/JS8_UI/WideGraph.cpp.obj
warning: 'void QDateTime::setTimeSpec(Qt::TimeSpec)' is deprecated: Use setTimeZone() instead [-Wdeprecated-declaration ]
  130 |       ts.setTimeSpec(Qt::UTC);
      |       ~~~~~~~~~~~~~~^~~~~~~~~
In file included from C:/Qt/6.9.3/mingw_64/include/QtCore/qcborvalue.h:11,
                 from C:/Qt/6.9.3/mingw_64/include/QtCore/qjsonvalue.h:8,
                 from C:/Qt/6.9.3/mingw_64/include/QtCore/qjsonobject.h:8,
                 from C:/Qt/6.9.3/mingw_64/include/QtCore/QJsonObject:1,
                 from C:/JTSDK64-Tools/tmp/wsjtx/JS8_Main/Message.h:10,
                 from C:/JTSDK64-Tools/tmp/wsjtx/JS8_Main/Inbox.h:8[ 98%] Building CXX object CMakeFiles/JS8Call.dir/JS8_Widgets/CheckableItemComboBox.cpp.obj
,
                 from C:\JTSDK64-Tools\tmp\wsjtx\JS8_UI\MessagePanel.h:3,
                 from C:\JTSDK64-Tools\tmp\wsjtx\JS8_UI\MessagePanel.cpp:1:
C:/Qt/6.9.3/mingw_64/include/QtCore/qdatetime.h:407:10: note: declared here
  407 |     void setTimeSpec(Qt::TimeSpec spec);
      |          ^~~~~~~~~~~
[ 99%] Building CXX object CMakeFiles/JS8Call.dir/JS8_Widgets/LazyFillComboBox.cpp.obj
[100%] Linking CXX executable JS8Call.exe

@Chris-AC9KH

Chris-AC9KH commented Feb 6, 2026

Copy link
Copy Markdown
Collaborator

@Joe-K0OG @rruchte yes, I didn't catch that because I wasn't watching compiler output during the builds. But I had fixed that before in other areas, and since it won't build anymore on Qt 6.10 without being fixed, it should be patched. It's a fairly simple fix 5a3dd7e

@rruchte

rruchte commented Feb 6, 2026

Copy link
Copy Markdown
Collaborator Author

Oops. Fixed in master. f2cb6c0

@Chris-AC9KH

Copy link
Copy Markdown
Collaborator

Thanks Rob, got rid of the compiler spam! :-)

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

Labels

enhancement New feature or request UI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants