D26983: [WIP]Enable adding place as first child

George Vogiatzis noreply at phabricator.kde.org
Wed Jan 29 16:43:51 GMT 2020


gvgeo added a comment.


  Although the target of this patch is the `else` part below, there is a need to add a place at the end.

INLINE COMMENTS

> kfileplacesmodel.cpp:1046
>  {
> -    addPlace(text, url, iconName, appName, QModelIndex());
> +    KBookmark bookmark = KFilePlacesItem::createBookmark(d->bookmarkManager, text, url, iconName);
> +

As it is, with a  `null QModelIndex`, the function below will move the new item to the top.

REPOSITORY
  R241 KIO

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

To: gvgeo, #frameworks, meven, broulik
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20200129/78eca7fc/attachment.html>


More information about the Kde-frameworks-devel mailing list