Alireza, can you look into fixing this. It does sound like a regression related to the new browser layout.

If it's not easy to fix at the browser level, it might make sense to add a button that gets enabled when there is a valid selection that says something like "Delete Selected Series".

Nicole, Alireza will add some notes here based on our earlier discussion with Steve on how to address this. He is leaving for vacation/grad school today, and unfortunately we were not able to intesect with you to discuss this issue in detail.

The remove button triggers deleting all selected series, studies and patients and that causes problems depending on where you've clicked and what got selected in addition to your click select.

Was the consensus to go with a right click menu or to add more options to the Remove button? Or both? It sounds like (from bug 3519) that the right click menu used to appear but with Delete disabled, right now on my Mac I don't get anything from a right click on a series (or study or patient).

yes, I think enabling remove button is the natural thing to do

I mean, enabling "remove" context menu

Sounds good, I'll continue that way and leave the remove button alone (will add some more tool tip hints though)

A stand alone test of the CTK DICOM Browser shows that a right click on the series table is detected, but when started from Slicer, it is not.

First pass at the CTK changes is here:
https://github.com/naucoin/CTK/commits/3792-unable-to-remove-one-series-of-an-exam
After building testing, you can run:
./CTK-build/bin/CTKDICOMWidgetsCppTests ctkDICOMBrowserTest1 ./CTKData/Data/DICOM/MRHEAD -I

Still debugging from the Slicer side, and doing futher testing on the CTK changes.

When I instantiate my updated CTK dicom browser via the python console:
db = ctk.ctkDICOMBrowser()
db.show()

the right click works.
If I manually trigger the right click context menu request on the Slicer version, it works:

dicomWidget = slicer.modules.DICOMWidget
dbrowser = dicomWidget.detailsPopup.dicomBrowser
tables = slicer.util.findChildren(dbrowser, 'dicomTableManager')[0]
seriesTable = findChildren(dbrowser, 'seriesTable')[0]
seriesTable.contextMenuPolicy
qp = qt.QPoint(3,4)
seriesTable.customContextMenuRequested(qp)

I can add connections inside the DICOM widget via python and those slots get triggered by the manual call, but it seems like the right click isn't getting through to the series table.

Digging more into the QSplitter as it's used to reorder the table widgets.

The code in DICOMWidgets.py for the class DICOMDetailsPopup create makes a new QWidget window and moves elements from the CTK browser into it. It moves the seriesTableView which doesn't respond to right click events. If I move the seriesTable into the new window, right clicks work again. Taking just the view left the table label, working on how to remove that from the Slicer version.

Got the changes localised to CTK, setting up the signals on the table view rather than the table. No changes required to Slicer, CTK pull request is here:
https://github.com/commontk/CTK/pull/587

Updated pull request branch with more description about the elements to be removed:
https://github.com/naucoin/CTK/commit/0a2bfec8eb3a3929d7441c8383222a7a986bcbc0

Pull request was merged to CTK:
https://github.com/commontk/CTK/pull/597
Pull request to update Slicer's CTK:
https://github.com/Slicer/Slicer/pull/340

Integrated in r24554
See http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=24554