Skip to content

[FIX] canvas/preview: Fix workflow preview rendering#2586

Merged
astaric merged 1 commit intobiolab:masterfrom
ales-erjavec:fixes/preview-render-svg-icons
Sep 13, 2017
Merged

[FIX] canvas/preview: Fix workflow preview rendering#2586
astaric merged 1 commit intobiolab:masterfrom
ales-erjavec:fixes/preview-render-svg-icons

Conversation

@ales-erjavec
Copy link
Contributor

Issue

Most widget icons are not displayed in the workflow preview due to QTBUG-63159

Description of changes

Reimplement QSvgGenerator.metric to report proper device pixel ratio.

Includes
  • Code changes
  • Tests
  • Documentation

Reimplement QSvgGenerator.metric to account for QTBUG-63159
Copy link
Member

@astaric astaric left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have tested with PyQt 4.11 and 5.6 and it fixes the problem for both.

return font


if QT_VERSION >= 0x50900 and \
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be pulled to the top of the file (after imports?)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would prefer to keep it near grab_svgwhere it is used.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@astaric astaric merged commit c95965b into biolab:master Sep 13, 2017
@ales-erjavec ales-erjavec deleted the fixes/preview-render-svg-icons branch November 14, 2017 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants