D27094: Multiple cell selection + new control element for entries + first version of selection actions

[Nikita Sirgienko]

sirgienko added inline comments.

INLINE COMMENTS

> asemke wrote in worksheet.cpp:1541
> unnecessary blank.

Where exactly?

> asemke wrote in worksheetcontrolelement.h:44
> these is* variables sound like function names. Let's consistently name private variables m_* like in WorksheetTextItem, etc.

Well, but this variables is public, so, I think, we could say, that they part of public interface.

REPOSITORY
  R55 Cantor

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

To: sirgienko, asemke
Cc: kde-edu, asemke, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20200202/98cb1b6a/attachment.html>