Summary
The TXDSP composite container has m_id = "tx_dsp" but its AppletEntry.id = "TXDSP", so dragging TXDSP's title bar produces MIME data "tx_dsp" which never matches the AppletEntry lookup in AppletPanel::dropEvent. Dragging TXDSP silently fails — no crash, no error, nothing happens.
Context
Identified during the triage of #1745 (crash when reordering applets with floating containers). That crash is being fixed in #1746, but the TXDSP mismatch was deliberately left out of scope — it's a separate, non-crash bug.
Expected behavior
TXDSP should be reorderable like every other applet in the panel.
Files involved
src/gui/AppletPanel.cpp — dropEvent lookup logic- Wherever TXDSP container sets its
m_id (suspected — the two IDs need to match)
Fix direction
Align the two IDs. Either:
- Change
m_id = "tx_dsp" → "TXDSP" in the TXDSP container, or
- Change
AppletEntry.id = "TXDSP" → "tx_dsp" in the registration site
Needs a quick check of which ID is persisted to settings to ensure we don't break saved applet ordering for existing users.
Summary
The TXDSP composite container has
m_id = "tx_dsp"but itsAppletEntry.id = "TXDSP", so dragging TXDSP's title bar produces MIME data"tx_dsp"which never matches theAppletEntrylookup inAppletPanel::dropEvent. Dragging TXDSP silently fails — no crash, no error, nothing happens.Context
Identified during the triage of #1745 (crash when reordering applets with floating containers). That crash is being fixed in #1746, but the TXDSP mismatch was deliberately left out of scope — it's a separate, non-crash bug.
Expected behavior
TXDSP should be reorderable like every other applet in the panel.
Files involved
src/gui/AppletPanel.cpp—dropEventlookup logicm_id(suspected — the two IDs need to match)Fix direction
Align the two IDs. Either:
m_id = "tx_dsp"→"TXDSP"in the TXDSP container, orAppletEntry.id = "TXDSP"→"tx_dsp"in the registration siteNeeds a quick check of which ID is persisted to settings to ensure we don't break saved applet ordering for existing users.