← Back to team overview

touch-packages team mailing list archive

[Bug 1480947] [NEW] Use SortFilterModel for custom models

 

Public bug reported:

While browsing the UITK documentation, I stumbled upon
https://developer.ubuntu.com/api/apps/qml/sdk-15.04/Ubuntu.Components.SortFilterModel/,
which is essentially QSortFilterProxyModel exposed to QML.

webbrowser-app has a number of custom models written in C++ that inherit
QSortFilterProxyModel. Some of them could most likely be replaced by a
pure QML component with this model from the UITK.

>From a very quick glance, the following models are candidates:
 - HistoryDomainModel
 - BookmarksFolderModel
 - HistoryDomainListChronologicalModel

This would have the advantage of removing a significant amount of custom
C++ code, speeding up build time and unit tests run time.

** Affects: webbrowser-app (Ubuntu)
     Importance: Medium
         Status: Triaged


** Tags: refactoring

** Changed in: webbrowser-app (Ubuntu)
       Status: New => Triaged

** Changed in: webbrowser-app (Ubuntu)
   Importance: Undecided => Medium

** Tags added: refactoring

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to webbrowser-app in Ubuntu.
https://bugs.launchpad.net/bugs/1480947

Title:
  Use SortFilterModel for custom models

Status in webbrowser-app package in Ubuntu:
  Triaged

Bug description:
  While browsing the UITK documentation, I stumbled upon
  https://developer.ubuntu.com/api/apps/qml/sdk-15.04/Ubuntu.Components.SortFilterModel/,
  which is essentially QSortFilterProxyModel exposed to QML.

  webbrowser-app has a number of custom models written in C++ that
  inherit QSortFilterProxyModel. Some of them could most likely be
  replaced by a pure QML component with this model from the UITK.

  From a very quick glance, the following models are candidates:
   - HistoryDomainModel
   - BookmarksFolderModel
   - HistoryDomainListChronologicalModel

  This would have the advantage of removing a significant amount of
  custom C++ code, speeding up build time and unit tests run time.

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/webbrowser-app/+bug/1480947/+subscriptions


Follow ups