D27523: Add an option to use a KUrlNavigator on the toolbar instead

Felix Ernst noreply at phabricator.kde.org
Thu Feb 20 16:00:41 GMT 2020


felixernst created this revision.
felixernst added a reviewer: VDG.
Herald added a project: Dolphin.
Herald added a subscriber: kfm-devel.
felixernst requested review of this revision.

REVISION SUMMARY
  This commit adds a showLocationBar action to toggle between
  using a location bar to navigate or using a new custom QWidgetAction.
  The KUrlNavigator of this new QWidgetAction synchonises itself with
  the KUrlNavigator of the active view whenever changes occur to either
  of them.
  
  When a separate location bar is already present and this QWidgetAction
  is on a toolbar it will act like an empty expanding spacer but will
  internally mimic the state of the KUrlNavigator of the active view.
  When no separate location bar is present setWidgetVisible() is used
  to show the internal KUrlNavigator. This switch is done by a
  QStackedWidget which is the defaultWidget() of the QWidgetAction.

REPOSITORY
  R318 Dolphin

BRANCH
  urlNavInToolBar (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D27523

AFFECTED FILES
  src/CMakeLists.txt
  src/dolphinmainwindow.cpp
  src/dolphinmainwindow.h
  src/dolphinui.rc
  src/dolphinurlnavigatorwidgetaction.cpp
  src/dolphinurlnavigatorwidgetaction.h
  src/dolphinviewcontainer.cpp
  src/dolphinviewcontainer.h
  src/settings/dolphin_generalsettings.kcfg

To: felixernst, #vdg
Cc: kfm-devel, pberestov, iasensio, fprice, MrPepe, fbampaloukas, alexde, Codezela, feverfew, meven, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20200220/6ac72188/attachment.htm>


More information about the kfm-devel mailing list