D24428: Fix document switcher plugin with multiple splitted view.

[Tristan Porteries]

tristanp created this revision.
Herald added a project: KDevelop.
Herald added a subscriber: kdevelop-devel.
tristanp requested review of this revision.

REVISION SUMMARY
  In differential D20548 <https://phabricator.kde.org/D20548> an issue with document switcher plugin and
  splitted views is described.
  The solution is to open/activate document instead of activating
  view (aka Sublime::View) as view are linked to area (aka Sublime::Area)
  and will show again in same area instead of current active area.
  
  But the differential solution was trying to retrieve the document
  from activated view which is not the perfect way as it must handle
  Sublime::UrlDocument to have a url for openDocument function.
  
  This patch use the document controller (aka IDocumentController)
  instead of sublime components.
  
  The plugin tracks a list of document (aka IDocument) and
  enables signals for open/activate/close actions.
  Every document is placed in list model via DocumentSwitcherItem
  class responsible of extracting text and icon from a document
  passed in its constructor.
  
  Without dependency to sublime, the plugin structure is
  simplified by not taking care about active main window, active
  area, and views. Only documents are tracking independently of the
  way they are displayed in UI.

REPOSITORY
  R32 KDevelop

BRANCH
  switch_doc_2

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

AFFECTED FILES
  plugins/documentswitcher/CMakeLists.txt
  plugins/documentswitcher/documentswitcheritem.cpp
  plugins/documentswitcher/documentswitcheritem.h
  plugins/documentswitcher/documentswitcherplugin.cpp
  plugins/documentswitcher/documentswitcherplugin.h

To: tristanp
Cc: kdevelop-devel, hmitonneau, christiant, glebaccon, domson, antismap, iodelay, alexeymin, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20191005/e56ee44c/attachment.html>