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

Update Plots.jl's colorbar_titel maps to PGFPlotsX's color bar style={ylabel} #3848

Conversation

jtschneider
Copy link
Contributor

@jtschneider jtschneider commented Sep 27, 2021

For most of the backends supported by Plots.jl the colorbar_title attribute maps to the attribute in the backend that ultimately is displayed on the vertical axis of the colorbar, which is vertically aligned as well.
However, this is not the case for PGFPlotsX as in this backend the title attribute is displayed on top of the colorbar while the ylabel is shown on the vertical bar.

This PR proposes this change for consistency and, admittedly very subjective, better looks.

Let me know if this is appreciated and/or desired.

Cheers
Jan

For most of the backends supported by Polts.jl the `colorbar_title` attribute maps to the attribute in the backend that ultimately is displayed on the vertical axis of the colorbar, which is vertically aligned. Not for PGFPlotsX as in this backend the `title` attribute is displayed on top of the colorbar while the `ylabel` is shown on the vertical bar.
@codecov
Copy link

codecov bot commented Sep 27, 2021

Codecov Report

Merging #3848 (801f1fb) into master (405bc08) will increase coverage by 0.01%.
The diff coverage is 100.00%.

❗ Current head 801f1fb differs from pull request most recent head 9cb8842. Consider uploading reports for the commit 9cb8842 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3848      +/-   ##
==========================================
+ Coverage   63.22%   63.23%   +0.01%     
==========================================
  Files          28       28              
  Lines        7491     7491              
==========================================
+ Hits         4736     4737       +1     
+ Misses       2755     2754       -1     
Impacted Files Coverage Δ
src/backends/pgfplotsx.jl 61.17% <100.00%> (ø)
src/pipeline.jl 90.59% <0.00%> (-0.05%) ⬇️
src/colorbars.jl 92.72% <0.00%> (+1.98%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 405bc08...9cb8842. Read the comment docs.

Copy link
Member

@BeastyBlacksmith BeastyBlacksmith left a comment

Choose a reason for hiding this comment

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

That looks reasonable to me, since this is your first contribution can you take a look at #3503 and if you like to, add your information to our .zenodo.json file?

@jtschneider
Copy link
Contributor Author

Hello @BeastyBlacksmith !
Thanks for the quick reply. I pulled the latest version and added my name to the contributor's list.

@BeastyBlacksmith BeastyBlacksmith merged commit 09a6622 into JuliaPlots:master Sep 27, 2021
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