[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: [kdevplatform] plugins/projectmanagerview: Enforce setting focus to the document tree view when
From:       Niko Sams <niko.sams () gmail ! com>
Date:       2013-12-24 10:40:17
Message-ID: CACQr-1xpEFwu2k02T+LRYHjxWw2Ug4uxTpaU8JwmE2_s6pgofQ () mail ! gmail ! com
[Download RAW message or body]

Seems to work well.
Thanks for that!

One thought: would it make sense to set the focus onto the editor on
doubleclick? that was the previous behavior.
(personally I don't care much)

Niko

On Mon, Dec 23, 2013 at 7:16 PM, Sven Brauch <svenbrauch@googlemail.com> wrote:
> Git commit 782ce8b5631c24762d19c9e309c9d927f6e09732 by Sven Brauch.
> Committed on 23/12/2013 at 18:16.
> Pushed by brauch into branch 'master'.
> 
> Enforce setting focus to the document tree view when opening a document
> 
> CCMAIL:niko.sams@gmail.com
> 
> M  +1    -0    plugins/projectmanagerview/projecttreeview.cpp
> 
> http://commits.kde.org/kdevplatform/782ce8b5631c24762d19c9e309c9d927f6e09732
> 
> diff --git a/plugins/projectmanagerview/projecttreeview.cpp \
> b/plugins/projectmanagerview/projecttreeview.cpp index dfd8969..d180e72 100644
> --- a/plugins/projectmanagerview/projecttreeview.cpp
> +++ b/plugins/projectmanagerview/projecttreeview.cpp
> @@ -284,6 +284,7 @@ void ProjectTreeView::slotActivated( const QModelIndex &index )
> {
> emit activateUrl( item->file()->url() );
> }
> +    setFocus();
> }
> //TODO test whether this could be replaced by \
> projectbuildsetwidget.cpp::showContextMenu_appendActions void \
> popupContextMenu_appendActions(QMenu& menu, const QList<QAction*>& actions)


[prev in list] [next in list] [prev in thread] [next in thread]