<div dir="ltr">Into digiKam, i do exactly a pure Qt5 port of KTabWidget to QTabWidget, especially about D&D.<div><br></div><div>Look my commit :<div><br></div><div><a href="https://quickgit.kde.org/?p=digikam.git&a=commit&h=11b5e18d24d80a1bf34fdcc78064a7cb8a26fabc">https://quickgit.kde.org/?p=digikam.git&a=commit&h=11b5e18d24d80a1bf34fdcc78064a7cb8a26fabc</a><br></div></div><div><br></div><div>Best</div><div><br></div><div>Gilles Caulier</div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-09-19 9:19 GMT+02:00 Thomas Lübking <span dir="ltr"><<a href="mailto:thomas.luebking@gmail.com" target="_blank">thomas.luebking@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Disclaimer: I've no knowledge about the made decision process/intentions.<br>
<br>
On Samstag, 19. September 2015 02:46:24 CEST, Jeremy Whiting wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
KTabWidget documentation on <a href="http://api.kde.org" rel="noreferrer" target="_blank">api.kde.org</a> it still says that KTabWidget is<br>
preferred over QTabWidget [1].<br>
</blockquote>
<br>
Wild guess: the class was moved, but nobody took a close look at the comments, so the API docu remained the same and on Qt4 level.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
In reading Qt documentation about drag and drop [2] it seems that you<br>
need to subclass widgets in order to specify any additional mime types<br>
that should be handled by a drop event<br>
</blockquote>
<br>
No, you can also install an eventfilter (on QTabBar/QTabWidget, the feature code was/is mostly in KTabBar, KTabWidget largely only forwards it)<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
(which we have in KTabWidget... so why not just use it...)<br>
</blockquote>
<br>
Probably because it contains much additional code and dropping mime is too rare - and usually only required from one location.<br>
The effort to bind the signals isn't much different from providing an eventfilter, QTabBar has ::tabAt(), so you already have the index for the event position.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Am I missing something? If not I suggest we reconsider and maybe<br>
move/copy?<br>
</blockquote>
<br>
"move". Having the same symbol in KWidgetAddons and kde4libssupport is gonna break things - badly.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
or maybe users of KTabWidget should just keep using KDELibs4Support?<br>
</blockquote>
<br>
I assume this is why the support lib exists - you can keep the widget until you ported away :-P<br>
<br>
Cheers,<br>
Thomas<br>
</blockquote></div><br></div>