auto

use Path::List (i.e. QVector!)

style: move down into an explicitly private section

const

afaik you can use m_paths.value(row, {})

|| !hasIndex(index.row(), index.size(), index.parent())

the above check makes this obsolete

auto

const

const

better not put it on the stack (could be deleted via dbus or funky stuff like that), but rather create on the heap and then set Qt::WA_DeleteOnClose on the dialog

the QUrl::fromLocalFile should not be required, Path(QString) does what you want, no?

this algorithm should be put somewhere out of this dialog, and get properly tested

also, the implementation looks pretty bad, performance wise. I think you want to put the includeName into an IndexedString and then use ProjectModel::item{,s}ForPath.

better use show and connect lambdas to signals that handle when the dialog got accepted

auto

auto