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

redesign of some glyphs for Shan #66

Closed
wants to merge 2 commits into from

Conversation

YusukeShirakihara
Copy link

Hello. Thank you for providing us with great fonts.
I’d like to submit a pull request regarding Noto Sans Myanmar for Shan language support.

About a year ago, SZTSIIT made the following issue #19 and pull request #63 about some incorrect characters, sizes and positioning of numbers for Shan in Noto Sans Myanmar.
Unfortunately, however, these topics were closed by SZTSIIT himself.

I’ve been working on fonts for Shan speaking people recently, and native speakers agreed with the changes in the request above.
Since the changes are desirable, I thought I might take over and submit this PR again.
In addition to SZTSIIT’s changes, we made a few adjustments.
And this PR is only for Noto Sans Myanmar.

Sample
ႁက္ႁည္ႁႁွႁုႁူႁႂႁ်ႂႁိုႁိူ
ၾက္ၾည္ၾၾွၾုၾူၾႂၾ်ႂၾိုၾိူ
ၽက္ၽည္ၽ
ꩡꩡ︀
႐႑႒႓႔႕႖႗႘႙
sample

Thank you for taking this request into consideration.

Copy link

google-cla bot commented Feb 28, 2024

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@simoncozens
Copy link
Contributor

Hello Shirakihara-san! Thank you so much for this contribution. There are a few questions:

  1. You will need to sign the CLA as mentioned above.
  2. I may ask someone to review the design; in my view the subscript FA looks quite squashed. I think probably all the glyphs want to keep the basic "circular" form.
  3. I am not sure if we really want smaller versions of subscripts underneath NNYA. I think we can probably fit full size subscripts there?

@YusukeShirakihara
Copy link
Author

YusukeShirakihara commented Feb 29, 2024

@simoncozens Thank you for your quick reply. Here are answers to each of your comments.

  1. You will need to sign the CLA as mentioned above.

I have signed the CLA and rescanned this request, but it has not been reflected... is there anything I can do?
スクリーンショット 2024-03-01 090025

  1. I may ask someone to review the design; in my view the subscript FA looks quite squashed. I think probably all the glyphs want to keep the basic "circular" form.

I agree with you. I updated fa_shn.sub and fa_shn.sub2. Could you check again?
sample-Regular

  1. I am not sure if we really want smaller versions of subscripts underneath NNYA. I think we can probably fit full size subscripts there?

That could be true, but it’s fine the way it is in Black. I think it would be better to adjust “blws” in the features.
sample-Black

@mi2ebi
Copy link

mi2ebi commented Mar 10, 2024

why is the subscript on က္ႁ a different size from the one on ည္ႁ? (even in whichever version my iphone has currently)

@simoncozens
Copy link
Contributor

I have signed the CLA and rescanned this request, but it has not been reflected... is there anything I can do?

I've worked out what is happening here; because you are using GitHub's "Keep my email address private" setting, you must change the email address in your CLA to your GitHub email (122098208+YusukeShirakihara​@users.noreply.github.com)

@YusukeShirakihara
Copy link
Author

YusukeShirakihara commented Mar 12, 2024

I see. I unchecked that setting. Finally all the checks passed. Sorry for the trouble.

@YusukeShirakihara
Copy link
Author

why is the subscript on က္ႁ a different size from the one on ည္ႁ? (even in whichever version my iphone has currently)

As you pointed out, in regular weight, both can be the same size, but in bolder weight such as black, the subscript ္ႁ seems too big for ည. I'm not sure, but that's may be the reason why Noto Sans Myanmar fonts are designed this way.

@mi2ebi
Copy link

mi2ebi commented Apr 8, 2024

the sizes are different even in regular, per the image above
image

@YusukeShirakihara
Copy link
Author

Sorry, my answer might have been a little confusing. Yes, in the current Regular design, the sizes are different. What I meant was that while it might be possible to make them the same size in Regular, it is probably difficult in bolder weights. But there may be other solution to this. It could be helpful if someone who knows about this font or the Shan language better provide us with further information on this.

@mi2ebi
Copy link

mi2ebi commented Apr 9, 2024

ahh okay yes. I feel like maybe consistent height & squishing them horizontally if needed could work

@YusukeShirakihara
Copy link
Author

This request is closed for now.

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