Skip to content
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

[ENH] Mosaic: Wrap legend #3866

Merged
merged 1 commit into from
Jun 14, 2019
Merged

Conversation

janezd
Copy link
Contributor

@janezd janezd commented Jun 7, 2019

Issue

Fixes #3052.

The function for wrapping the legend is put into module plotutil, so it can also be used elsewhere.

TODO: When decreasing the widget height, the legend can fall out of view at the bottom, despite my attempts to correctly compute the maximal square size.

Includes
  • Code changes
  • Tests
  • Documentation

@janezd janezd force-pushed the owmosaic-wrap-legend branch from 6ba3a48 to 1098e8d Compare June 7, 2019 11:49
@codecov
Copy link

codecov bot commented Jun 7, 2019

Codecov Report

Merging #3866 into master will decrease coverage by <.01%.
The diff coverage is 79.16%.

@@            Coverage Diff             @@
##           master    #3866      +/-   ##
==========================================
- Coverage   84.97%   84.96%   -0.01%     
==========================================
  Files         378      378              
  Lines       70496    70520      +24     
==========================================
+ Hits        59905    59920      +15     
- Misses      10591    10600       +9

@janezd janezd force-pushed the owmosaic-wrap-legend branch from 1098e8d to d23b8ac Compare June 7, 2019 12:13
@janezd janezd changed the title OWMosaic: Wrap legend [ENH] OWMosaic: Wrap legend Jun 14, 2019
@janezd janezd changed the title [ENH] OWMosaic: Wrap legend [ENH] Mosaic: Wrap legend Jun 14, 2019
@janezd janezd force-pushed the owmosaic-wrap-legend branch from d23b8ac to d8f6cf6 Compare June 14, 2019 07:53
@janezd janezd force-pushed the owmosaic-wrap-legend branch from d8f6cf6 to 96361c6 Compare June 14, 2019 09:05
@PrimozGodec PrimozGodec merged commit 0844614 into biolab:master Jun 14, 2019
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.

Mosaic Display: legend falls out of view
2 participants