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

Christoph Cullmann <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Latest Commit|                            |https://invent.kde.org/util
                   |                            |ities/kate/commit/9880babc6
                   |                            |9472a97dd6d095dd39205099d6d
                   |                            |4995
             Status|ASSIGNED                    |RESOLVED

--- Comment #8 from Christoph Cullmann <[email protected]> ---
Git commit 9880babc69472a97dd6d095dd39205099d6d4995 by Christoph Cullmann, on
behalf of Waqar Ahmed.
Committed on 17/01/2022 at 19:28.
Pushed by cullmann into branch 'master'.

Improve views closing in split view behaviour

Currently if you have a doc that has 2 views in 2 different splits:

A | A

and you close one of them, it closes the other as well. This behaviour
is really non-intuitive and is very unexpected to say the least. To
start with, no one asked kate to close the other view so it shouldn't
auto close it.

Moreover suppose you have a viewspace with:

AB

and you want to open a new split, the new split will open to the *left*
of the current one resulting in: A | AB, instead of AB | A. This is
the opposite of what any app does.

With this change, I am proposing a different solution:
- If a doc has multiple views, only close the one that the user asked
  for in the current viewspace.
- If the viewspace becomes empty, there is no need to refill it, close
  the viewspace. If this is the last viewspace, it remains open.
- Lastly, when doing vertical split, open the new viewspace *after* the
  current one not before.

This behaviour is consistent with other editors and apps doing splits.

M  +14   -4    kate/kateviewmanager.cpp
M  +3    -1    kate/kateviewmanager.h
M  +29   -1    kate/kateviewspace.cpp
M  +7    -0    kate/kateviewspace.h

https://invent.kde.org/utilities/kate/commit/9880babc69472a97dd6d095dd39205099d6d4995

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

Reply via email to