D11118: Show a message if Konsole part is not installed

Elvis Angelaccio noreply at phabricator.kde.org
Wed Mar 7 22:41:55 GMT 2018


elvisangelaccio added inline comments.

INLINE COMMENTS

> terminalpanel.cpp:142-143
> +        } else if (!m_konsolePartMissingLabel) {
> +            m_konsolePartMissingLabel = new QLabel(i18n(
> +                    "Terminal cannot be shown because Konsole part (package \"konsole-part\" or \"konsole-kpart\") is not installed."));
> +            m_konsolePartMissingLabel->setAlignment(Qt::AlignCenter);

How about using a KMessageWidget instead? (see e.g. https://git.reviewboard.kde.org/r/130007/)

REPOSITORY
  R318 Dolphin

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

To: rominf, #dolphin
Cc: elvisangelaccio, broulik, progwolff, #dolphin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180307/2cdf3644/attachment.htm>


More information about the kfm-devel mailing list