Skip to content

Conversation

@seisman
Copy link
Member

@seisman seisman commented Dec 7, 2025

Description of proposed changes

Add missing aliases for -B and -S.

Related to #4249.

Upstream documentation: https://docs.generic-mapping-tools.org/6.6/legend.html#s

Preview: https://pygmt-dev--4254.org.readthedocs.build/en/4254/api/generated/pygmt.Figure.legend.html

@seisman seisman added this to the 0.18.0 milestone Dec 7, 2025
@seisman seisman added enhancement Improving an existing feature needs review This PR has higher priority and needs review. labels Dec 7, 2025
scale: float | None = None,
position="JTR+jTR+o0.2c",
box: Box | bool = False,
projection: str | None = None,
Copy link
Member Author

Choose a reason for hiding this comment

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

I also moved the common options to the end of the parameter list.

@seisman seisman mentioned this pull request Dec 7, 2025
19 tasks
@seisman seisman force-pushed the legend/scale-frame branch from bbb229b to 3e5754c Compare December 7, 2025 03:26
@seisman seisman added final review call This PR requires final review and approval from a second reviewer and removed needs review This PR has higher priority and needs review. labels Dec 8, 2025
@seisman seisman removed the final review call This PR requires final review and approval from a second reviewer label Dec 8, 2025
@seisman seisman merged commit b1737f7 into main Dec 8, 2025
25 checks passed
@seisman seisman deleted the legend/scale-frame branch December 8, 2025 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Improving an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants