Review Request 123878: Introduction of a Concentration mode

[Milian Wolff]

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

Ship it!


you are still missing the new meta+ shortcuts for the assistants, otherwise this lgtm.

some questions on my part, which should not hold this nice patch up any longer, just would like to start a discussion on them:
- do we really want to hide/replace the tab bar? I'm "one of those guys" and I still use the tab bar from time to time - esp. together with the shortcuts to go to the left/right document (alt + shift + left/right). note how this differs from the documenttabwidget, as it always quickly switches between buddies e.g. or between neighbors that I can reorder manually if I want to.
- all shortcuts with an ALT in them, like the above, trigger the issue of showing the flickering menu (when the user first presses alt, then shift or anything else) - could we maybe instead show the menu floating above the toolbar? this sounds a bit crazy, but if its doable without too much effort it would be nice. this way, no ui items would be pushed down and no flickering would occur. Heck - we could go even so far as to push the actions into a QMenu which gets shown centered in the window when alt is pressed (similar to the cycle action for next/prev used document)...
- could we show a (passive?) message somewhere that concentration mode was enabled and you can get out via $shortcut?

thanks! I'll keep this applied locally and test at work tomorrow.


sublime/mainwindow_p.cpp (line 50)
<https://git.reviewboard.kde.org/r/123878/#comment56006>

    missing:
    
       , m_timer(nullptr)



sublime/mainwindow_p.cpp (line 102)
<https://git.reviewboard.kde.org/r/123878/#comment56005>

    QStringLiteral



sublime/mainwindow_p.cpp (line 208)
<https://git.reviewboard.kde.org/r/123878/#comment56004>

    style: place * next to type



sublime/mainwindow_p.cpp (line 455)
<https://git.reviewboard.kde.org/r/123878/#comment56007>

    indeed :P


- Milian Wolff


On June 7, 2015, 9:51 a.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123878/
> -----------------------------------------------------------
> 
> (Updated June 7, 2015, 9:51 a.m.)
> 
> 
> Review request for KDevelop and KDE Usability.
> 
> 
> Repository: kdevplatform
> 
> 
> Description
> -------
> 
> One of the perceived problems in KDevelop that affect me the most is that we're visually heavy. And we are. This patch doesn't address this, but I believe it will start to give us some hints on where we want to go.
> 
> This patch, basically adds a new togglable state to the main window, which allows our user to reduce some of the noise in the UI, allowing him to concentrate on what he came to do. This has a big advantage: we get to assume that the user is familiar with the shortcuts.
> What it does:
> 
> * It hides all the toolview handles (not the toolviews).
> * It hides the menu bar. Since I realized I was using Alt+* to access some menus, it's showing the menu when Alt is pressed.
> * It adds a toolbar that lets the user get out of the concentration mode, and also receives the menu widget (with the working sets and area button).
> * Also since I realized I was using some other shortcuts, I took the liberty to adapt them, I think they're far easier to remember than the previous ones.
> * Replaces the container tab bar to work without it (it has a label instead, long story, I know).
> 
> Now it works reasonably well, you can see the screenshot if you want to get an idea without trying the patch. I'm still undecided about whether we want to hide them.
> 
> 
> Diffs
> -----
> 
>   plugins/contextbrowser/browsemanager.cpp 70a3125 
>   sublime/mainwindow.cpp a1acdf3 
>   sublime/mainwindow_p.h 5477091 
>   sublime/mainwindow_p.cpp aa9e10e 
> 
> Diff: https://git.reviewboard.kde.org/r/123878/diff/
> 
> 
> Testing
> -------
> 
> I've been using it for work during the last few days. It doesn't crash (more) and well, I don't miss most of the things, so it's probably safe to get this in as is.
> 
> 
> File Attachments
> ----------------
> 
> Small Window
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/05/22/f4ee28f6-1e0c-41bc-8d7a-b216d23ca5f4__kdevelop-concentration.png
> Concentration Mode
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/05/22/62c102b1-dbc7-4bd3-9e96-7b866709ca3b__kdevelop-concentration-1.png
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20150622/7697d6e5/attachment-0001.html>