From a6403716439ae72bfdf86a14af94ea9d212f08ee Mon Sep 17 00:00:00 2001 From: Alexander Saoutkin Date: Mon, 2 Sep 2019 23:15:08 +0200 Subject: Fixing bugs in new folders in tabs feature Summary: Fixing bug where urls in secondary view containers would not be considered for the open new folders in tabs feature. Test Plan: Manual testing. Testing for no regressions. Testing that URL is found if in secondary view container Reviewers: elvisangelaccio Reviewed By: elvisangelaccio Subscribers: kfm-devel, ngraham Tags: #dolphin Differential Revision: https://phabricator.kde.org/D23655 --- src/dolphintabwidget.h | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'src/dolphintabwidget.h') diff --git a/src/dolphintabwidget.h b/src/dolphintabwidget.h index 3e8301725..7eb001b21 100644 --- a/src/dolphintabwidget.h +++ b/src/dolphintabwidget.h @@ -80,9 +80,13 @@ public: /** * @param url The URL that we would like - * @return index of the tab with the desired URL. returns -1 if not found + * @return a QPair with first containing the index of the tab with the + * desired URL or -1 if not found. Second says true if URL is in primary + * view container, false otherwise. False means the URL is in the secondary + * view container, unless first == -1. In that case the value of second + * is meaningless. */ - int getIndexByUrl(const QUrl& url) const; + QPair getIndexByUrl(const QUrl& url) const; signals: /** -- cgit v1.3