-
-
Notifications
You must be signed in to change notification settings - Fork 86
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Topic Modeling: Base selection style on itemdelegates #751
Conversation
Codecov Report
@@ Coverage Diff @@
## master #751 +/- ##
==========================================
+ Coverage 74.30% 74.48% +0.17%
==========================================
Files 73 74 +1
Lines 9669 9896 +227
Branches 1321 1334 +13
==========================================
+ Hits 7185 7371 +186
- Misses 2229 2269 +40
- Partials 255 256 +1 |
painter.setPen( | ||
QtGui.QPen(text_color_for_state(option.palette, option.state))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should probably be
ctx.palette.setColor(QtGui.QPalette.Text, text_color_for_state(option.palette, option.state))
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
28b280b
to
bb325aa
Compare
This one needs at least orange-widget-base 4.14.0. So we need to increase the minimum version to |
66e9830
to
1a568e0
Compare
…ates summaries but Table not
498bdbd
to
f62b8d9
Compare
I added two commits to this branch. First increases minimal version of orange-widget-base. The second fixes summary implementation which does not work when a version of orange-widget-base >=4.14 and the version of Orange==3.28. |
Issue
Fixes #711.
Description of changes
Use
text_color_for_state
to color selection in Topic Modeling.Includes