Review Request: remove split view functionality

[Sven Langkamp]

> On July 18, 2011, 3:19 p.m., Sven Langkamp wrote:
> > Having two windows side by side introduces lots of other problem:
> > -It would bring back the jumping when switching views, because dockers would be hidden and shown all the time.
> > -You can't have the dockers on one side of the screen as there are two different windows.
> > -You need a good window manager, so that on splitting the windows appear side by side.
> > 
> > Some of the problems with the current system are there because we tried to avoid those problems e.g. we switch the dockers internally to avoid the jumping. The problem you pointed out a purely implementation problems and not problems with the design of the UI. I don't think that we should change the UI just because the implementation isn't working.
> 
> Boudewijn Rempt wrote:
>     Yes, I agree in principle... But the design limits the usefulness because we cannot have two different documents in one window in any case, and given the problems we have had over the years with this feature I doubt that we will manage to have it fixed. And there are so many crashes and every new docker makes it likely we'll get one more crash that I don't think trying to fix this ancient code is worth it.  It's not like other apps all have this feature -- LibreOffice doesn't. But I'd welcome a patch that really fixes split view -- though whether that should be based on the current, very ancient, code, I don't know.
> 
> Boudewijn Rempt wrote:
>     Interestingly, MyPaint is adding split view functionality... But they do it sort of right: they can have both vertical and horizontal splits in one window. I'm wondering whether it wouldn't be a good idea to make a fresh start for Calligra, with something that allows all those missing features.

As far as I can see MyPaint does a few things differently. Views only control how the image is display, everything else it shared between the views. In Calligra there is a hard seperation and only the document itself is shared between the view. I wonder if it wouldn't be better to share at least some (or maybe everything) things between the views.

The main problem is the old KPart approach to do the splitting. That really inflexible and doesn't allow us to fix some of the problem that we are currently facing.


- Sven


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101993/#review4819
-----------------------------------------------------------


On July 18, 2011, 1:03 p.m., Boudewijn Rempt wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101993/
> -----------------------------------------------------------
> 
> (Updated July 18, 2011, 1:03 p.m.)
> 
> 
> Review request for Calligra.
> 
> 
> Summary
> -------
> 
> This patch removes the buggy split-view functionality from KoMainWindow. split-view has a number of problems:
> 
> * crashes when removing views
> * duplicates the actions so when a view is split you cannot e.g. delete a shape with the delete key
> * it is limited in that it can only show views on one document, instead of splitting the view and then making it possible to show several documents side-by-side
> 
> After discussion on irc, we felt it was better to let the window manager handle side-by-side views. To that end, a patch to optionally hide dockers if the window loses focus will follow.
> 
> 
> This addresses bugs 239309, 265200, 277953, and 278000.
>     http://bugs.kde.org/show_bug.cgi?id=239309
>     http://bugs.kde.org/show_bug.cgi?id=265200
>     http://bugs.kde.org/show_bug.cgi?id=277953
>     http://bugs.kde.org/show_bug.cgi?id=278000
> 
> 
> Diffs
> -----
> 
>   flow/part/flow.rc bd2913a 
>   karbon/data/karbon.rc 18c7156 
>   kpresenter/part/stage.rc d48fd3c 
>   kpresenter/stage.rc 3e0f53d 
>   krita/krita.rc 3d5d85d 
>   libs/main/KoDocument.cpp 515911a 
>   libs/main/KoMainWindow.h d44aff6 
>   libs/main/KoMainWindow.cpp c629827 
>   tables/tables.rc 0fde1b6 
>   words/part/words.rc 4e10d36 
> 
> Diff: http://git.reviewboard.kde.org/r/101993/diff
> 
> 
> Testing
> -------
> 
> Started several apps, opened and closed documents.
> 
> 
> Thanks,
> 
> Boudewijn
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20110722/5a1125a6/attachment.htm>