Exporting KFilePlaceEditDialog

[Elvis Angelaccio]

On Sun, Feb 19, 2017 at 6:51 PM, Albert Astals Cid <aacid at kde.org> wrote:
> El diumenge, 19 de febrer de 2017, a les 13:02:24 CET, Elvis Angelaccio va
> escriure:
>> Hi,
>> currently KFilePlaceEditDialog is not exported by kfilewidgets. As I
>> understand it, it's supposed to be used only through KFilePlacesView
>> (that's what gwenview and konqueror do).
>>
>> However, Dolphin is using its own implementation of
>> KFilePlacesModel/KFilePlacesView. This is why Dolphin has its own
>> dialog to edit places, which is exactly the same as
>> KFilePlaceEditDialog but with more bugs, e.g. see [1].
>>
>> Could we make KFilePlaceEditDialog a public class? I don't really see
>> downsides in doing this. We reduce code duplication (by killing the
>> buggy Dolphin dialog) and Dolphin can still use its own model/view
>> places stuff (I tested it localy and seems to work).
>
> The other option is improving KFilePlacesModel/KFilePlacesView so that Dolphin
> can use it.
>
> Without knowing about them (which i don't) can't say what's better.
>
> Do you know what is stopping Dolphin using KFilePlacesModel/KFilePlacesView ?

The timeline:/ and search:/ urls mostly. There is also a grouping
feature but I'm not sure how that works.
Though I agree that in the long-term it would be better to improve the
upstream KFilePlacesModel/View...

>
> Cheers,
>   Albert
>
>>
>> Cheers,
>> Elvis
>>
>> [1]: https://bugs.kde.org/show_bug.cgi?id=376619
>
>