Review Request 126222: Fix rating widget flickering when selecting rating with mouse

Xuetian Weng wengxt at gmail.com
Fri Dec 4 16:26:04 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126222/
-----------------------------------------------------------

(Updated Dec. 4, 2015, 4:26 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks, David Faure and Lamarque Souza.


Changes
-------

Submitted with commit efb14cb69ad53bcdb4f3f7b4f2439183d27ebb05 by Weng Xuetian to branch master.


Repository: kwidgetsaddons


Description
-------

Refactor the rating calculation:
1. Disable feature introduced in bug 171343 when halfSteps is enabled.
2. Merge the two pieces of code, and try to make it clearer and simpler.
3. make hover 0 star also show visual effect.


Diffs
-----

  src/kratingpainter.cpp 1a4378a 
  src/kratingwidget.cpp d0d2903 

Diff: https://git.reviewboard.kde.org/r/126222/diff/


Testing
-------

Current rating is 0, hover on the 1st star, 1 star lights up, click on 1st star, rating becomes 2.

Current rating is 2. hover on the 1st star, half star lights up, click on 1st star, rating becomes 1.

Current rating is 1/2. move mouse to the left most edge, star's color becomes lighter. Click and rating becomes 0.


Thanks,

Xuetian Weng

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20151204/dbd3ffd4/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list