Review Request 129773: Escape closes fullscreen

[Hartmut Brüning]

> On Jan. 4, 2017, 10:07 nachm., Albert Astals Cid wrote:
> > I honestly don't think this is a good idea, all the KDE apps that have full screen go into it with Ctrl+Shift+F and out of it by the same, adding Esc just adds inconsistency to it.

Yes, it is a good idea. Believe me, when I left my laptop in okular fullscreen mode, my wife did not find a way out of it. She did not get the idea to press the right mouse button, because the laptop has a touchpad with softkeys and there is no physical right mouse button.

The most obvious in that case is to exit fullscreen mode with escape. Not everybody is willing or able to learn key combinations.

If the menu were visible, I would agree. But the menu is not visible. In okular fullscreen mode is more like presentation mode. And presentation mode can be closed with escape. So I would expect consistency with fullscreen mode and presentation mode.

And in every sense, beeing able to close fullscreen mode with escape really does not harm anything. I expect from a good piece of software that it guides me where I should better be guided and that it supports me and anticipates what I want to do. I want to use my software in many ways. Sometimes only with mouse, sometimes only with keyboard, sometimes only with touchscreen. Would be sad very for if kde is not willing to support me beeing flexible. 

Could you please tell me your opinion about bug 269677, too. I am currently implementing this one. The hint to Ctrl+M at the end of this discussion is a joke, isnt't it?


- Hartmut


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129773/#review101802
-----------------------------------------------------------


On Jan. 4, 2017, 4:35 nachm., Hartmut Brüning wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129773/
> -----------------------------------------------------------
> 
> (Updated Jan. 4, 2017, 4:35 nachm.)
> 
> 
> Review request for Okular.
> 
> 
> Bugs: 366276
>     http://bugs.kde.org/show_bug.cgi?id=366276
> 
> 
> Repository: okular
> 
> 
> Description
> -------
> 
> I introduced a m_exitFullScreenAction that listens on escape key. It is active in full screen mode only. I tested that escape key in menu or dialogs are not affected by this change.
> 
> 
> Diffs
> -----
> 
>   shell/shell.h d015b3fc 
>   shell/shell.cpp 448b9ff7 
> 
> Diff: https://git.reviewboard.kde.org/r/129773/diff/
> 
> 
> Testing
> -------
> 
> yes:
> 
> press escape in normal mode
>     -> nothing happens -> ok
> clicked into menu, press escape
>     -> menu closes -> ok
> opened configure menu, press escape
>     -> configure menu closes -> ok
> opened file dialog, press escape
>     -> file dialog closes -> ok
> switched to fullscreen, press escape
>     -> fullscreen closes -> ok
> switched to fullscreen, right click, close fullscreen
>     -> fullscreen closes -> ok
> switched to fullscreen, right click, press escape, press escape
>     -> context menu opens, closes and fullscreen closes -> ok
> press Ctrl+Shift+F, press Ctrl+Shift+F -> fullscreen opens and closes
>     -> ok
> press Ctrl+Shift+F, press escape -> fullscreen opens and closes
>     -> ok
> switch to fullscreen, terminate okular, start reokular, press escape
>     -> okular starts in fullscreen and closes fullscreen -> ok
> 
> 
> Thanks,
> 
> Hartmut Brüning
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20170105/605226ad/attachment.html>