[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]