D17494: Allow translation of the style plugin names in molecule viewer

Yuri Chornoivan noreply at phabricator.kde.org
Sat Dec 15 18:56:02 GMT 2018


yurchor marked 2 inline comments as done.
yurchor added inline comments.

INLINE COMMENTS

> pino wrote in moleculeview.cpp:173
> this now uses an UserRole...

Sure. Does not work when changed to Qt::DisplayRole

> pino wrote in moleculeview.cpp:176
> ... but this comparison is still with a DisplayRole, so it will never match

But it does (tested). Does not work when changed to Qt::UserRole.

REPOSITORY
  R326 Kalzium

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

To: yurchor, #kde_edu, pino
Cc: pino, kde-edu, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20181215/3e3180a9/attachment.html>


More information about the kde-edu mailing list