Just a couple of tips:
1) when referencing model data in a delegate it is usually helpful for later readability to make that referencing more explicit.
E.g.. instead of using just "serverName" write "model.serverName"
This makes it more obvious which data is local to the delegate and which data is from the model
2) this is probably just because this is more an example code to demonstrate the problem, but qsTr(serverName) does not make any sense. "serverName" is a property from the model. if is contains translatable text it has had to be translated at its source already.
Cheers,
_
Bookmarks