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

Add figure space as style_positive option #366

Merged
merged 3 commits into from
Nov 6, 2023

Conversation

bwiernik
Copy link
Contributor

This PR adds a third option for style_positive. If style_positive = "space", positive values are prepended by \u2007 (see https://www.fileformat.info/info/unicode/char/2007/index.htm).

\u2007 is a "figure space" -- a blank space the same width as a digit or + or proper minus sign. Compared to using style_positive = "none", prepending a figure space to positive numbers can help ensure that positive and negative numbers are consistently aligned, especially when they are left- or right-justified.

@thomasp85
Copy link
Member

Can I get you to update the PR to remove conflicts?

@bwiernik
Copy link
Contributor Author

bwiernik commented Nov 1, 2023

I rebased the PR and the conflicts are resolved now.

@bwiernik
Copy link
Contributor Author

bwiernik commented Nov 1, 2023

The failing check appears to be unrelated to the PR (it's in label_dollar())

@thomasp85
Copy link
Member

Thanks - can I get you to add a news bullet

@thomasp85 thomasp85 added this to the scales 1.3.0 milestone Nov 2, 2023
@thomasp85 thomasp85 merged commit 33d0e97 into r-lib:main Nov 6, 2023
12 checks passed
@thomasp85
Copy link
Member

Thanks!

@bwiernik bwiernik deleted the figure-space branch November 30, 2023 20:15
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