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

Small changes latin #60

Merged
merged 5 commits into from
Jun 23, 2022
Merged

Small changes latin #60

merged 5 commits into from
Jun 23, 2022

Conversation

RosaWagner
Copy link
Contributor

@RosaWagner RosaWagner commented Jun 1, 2022

Move Paragraph and Section symbols to Latin_Plus, to preserve kernel as the most minimalist set possible.
@vv-monsalve I think this was your initial proposal, do you still agree?

@RosaWagner
Copy link
Contributor Author

friendly ping @vv-monsalve

I have hesitation in putting section and paragraph into Latin Core instead of Latin Plus since these are used in legal documents.

But I am definitely in favour of removing them from Kernel.

Lib/glyphsets/data.json Outdated Show resolved Hide resolved
Lib/glyphsets/data.json Outdated Show resolved Hide resolved
@vv-monsalve
Copy link
Contributor

friendly ping @vv-monsalve

Hi, thanks for the ping, I didn't receive the first notification.

I don't recall proposing to move Paragraph but Section. Although from your comment about legal documents, perhaps it would be needed to leave them both in the Latin Core.

In that case, it would be nice to include them in some learn section in the GF-guide about their design as they tend to be of low quality. We've talked about this for a few characters now. Maybe it could be at least a list with links to keep it simple.

But I am definitely in favour of removing them from Kernel.

Agree

@RosaWagner RosaWagner merged commit 8ccf242 into googlefonts:main Jun 23, 2022
@RosaWagner RosaWagner deleted the small-changes-latin branch June 23, 2022 12: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.

3 participants