https://bugs.kde.org/show_bug.cgi?id=401464

Eike Hein <h...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|REPORTED                    |RESOLVED
      Latest Commit|                            |https://commits.kde.org/pla
                   |                            |sma-desktop/dcb4c158b41452d
                   |                            |18e629b30b472b6d3294070ef

--- Comment #5 from Eike Hein <h...@kde.org> ---
Git commit dcb4c158b41452d18e629b30b472b6d3294070ef by Eike Hein, on behalf of
Oleg Solovyov.
Committed on 02/04/2019 at 09:49.
Pushed by hein into branch 'master'.

foldermodel: store information about screen used

Summary:

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

Reviewers: broulik, hein, #plasma

Reviewed By: hein, #plasma

Subscribers: cfeck, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D18979

M  +15   -14   containments/desktop/plugins/folder/foldermodel.cpp
M  +1    -0    containments/desktop/plugins/folder/foldermodel.h

https://commits.kde.org/plasma-desktop/dcb4c158b41452d18e629b30b472b6d3294070ef

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to