D19988: Fix lengthy URLs in tooltips

Elvis Angelaccio noreply at phabricator.kde.org
Sun Mar 24 09:33:37 GMT 2019


elvisangelaccio added inline comments.

INLINE COMMENTS

> widgetfactory.cpp:131-132
> +                auto labelString = valueString;
> +                if (labelString.size()>80) {
> +                    labelString = KStringHandler::csqueeze(labelString, 80);
> +                }

Please put the `80` constant in a variable.

Please add spaces before/after the `>` operator.

REPOSITORY
  R824 Baloo Widgets

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

To: krutovmikhail, #dolphin
Cc: elvisangelaccio, broulik, #baloo, gennad, domson, ashaposhnikov, astippich, spoorun, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20190324/8e27e000/attachment.htm>


More information about the kfm-devel mailing list