D15775: Make the item background color and page cache properties available from View component

Anthony Fieroni noreply at phabricator.kde.org
Thu Oct 4 12:38:59 BST 2018


anthonyfieroni added a comment.


  Also can  you add parentheses over single line code and if you know better way to get impl() without duplicate calls will be great :)

INLINE COMMENTS

> View.cpp:160
>          canvas->setGeometry(0, 0, q->width(), q->height());
> +        static_cast<DocumentImpl*>(document->impl())->setBackgroundColor(q->fillColor());
> +        static_cast<DocumentImpl*>(document->impl())->setPageCache(m_cache);

You don't check against impl is not nullptr?

REPOSITORY
  R8 Calligra

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

To: dcaliste, leinir, danders, anthonyfieroni, #calligra:_3.0
Cc: boemann, Calligra-Devel-list, dcaliste, cochise, vandenoever
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20181004/7f3b4589/attachment.htm>


More information about the calligra-devel mailing list