D16648: Open externally called files/directories in new tabs

Alexander Saoutkin <[email protected]> Tue, 21 May 2019 13:48:47 +0000

Newsgroups	gmane.comp.kde.devel.kfm,gmane.comp.kde.devel.kwin
Message-ID	<[email protected]>

feverfew added a comment.


  In D16648#467854 <https://phabricator.kde.org/D16648#467854>, @fvogt wrote:
  
  > It looks like several QUrls were changed to QString here, why? Most of those QStrings, if not all, actually contain URLs.
  
  
  If I remember correctly Qt methods that are publicised on DBus cannot have parameters as QList<QUrl>, hence I had to use QStringList. This seems to happen often enough that Qt even has helper functions to do the conversion. There are some more conversions to avoid unnecessarily changing method signatures of existing functions.

REPOSITORY
  R318 Dolphin

REVISION DETAIL
  https://phabricator.kde.org/D16648

To: feverfew, #dolphin, elvisangelaccio
Cc: fvogt, fikrim, magar, fbg13, davidedmundson, kwin, ngraham, elvisangelaccio, anthonyfieroni, kfm-devel, alexde, feverfew, meven, spoorun, navarromorales, firef, andrebarros, emmanuelp, mikesomov