D29871: Enable option to show hidden folders in sidebar, plus small sidebar code fixes (eg: scrollbar display)

David Faure noreply at phabricator.kde.org
Wed Aug 5 16:50:12 BST 2020


dfaure added inline comments.

INLINE COMMENTS

> sidebar_widget.cpp:578
> +                if (currentButtonInfo().canToggleShowHiddenFolders) {
> +                    QAction *m_ToggleShowHiddenFolders;
> +                    m_ToggleShowHiddenFolders = buttonPopup->addAction(i18n("Show Hidden Folders..."), this, SLOT(slotToggleShowHiddenFolders()));

The naming m_ (followed by lowercase, BTW) is for member variables.

Please s/m_T/t/ and join with the next line.

REPOSITORY
  R226 Konqueror

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

To: rrosch, dfaure
Cc: kde-doc-english, gennad, fbampaloukas, skadinna
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-doc-english/attachments/20200805/e0a23317/attachment.htm>


More information about the kde-doc-english mailing list