D11740: ktooltipwidget: Subtract margins from available size

[Christoph Feck]

cfeck added inline comments.

INLINE COMMENTS

> ktooltipwidget.cpp:183
>      d->addWidget(content);
> +    const auto marginSize = QSize{
> +        2 * content->style()->pixelMetric(QStyle::PM_DefaultTopLevelMargin),

Call me old-school, but I prefer to see the type before the variable name, i.e.

  const QSize marginSize(w, h);

instead of

  const auto marginSize = QSize(w, h);

One word less to read.

REPOSITORY
  R236 KWidgetsAddons

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

To: michaelh, #frameworks, elvisangelaccio
Cc: elvisangelaccio, cfeck, michaelh, ngraham
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180330/1bf1d5a8/attachment.html>