D18979: foldermodel: store information about screen used

Oleg Solovyov noreply at phabricator.kde.org
Wed Feb 13 12:12:48 GMT 2019


McPain created this revision.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
McPain requested review of this revision.

REVISION SUMMARY
  BUG: 401464
  
  According to FolderModel::filterAcceptsRow code, it is possible that folderview is used by containment but not associated with a screen.
  In case of m_screen == -1 there was no information about which items we should filter out.
  
  Instead, we do not change m_screen to -1 and we can filter out items that does not belong to our screen.
  m_screenUsed shows whether folderview is associated with a screen or not

REPOSITORY
  R119 Plasma Desktop

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

AFFECTED FILES
  containments/desktop/plugins/folder/foldermodel.cpp
  containments/desktop/plugins/folder/foldermodel.h

To: McPain
Cc: plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190213/e53c79a6/attachment.html>


More information about the Plasma-devel mailing list