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

Fix Whalebird font stack #732

Merged
merged 3 commits into from Nov 21, 2018
Merged

Fix Whalebird font stack #732

merged 3 commits into from Nov 21, 2018

Conversation

amyspark
Copy link
Contributor

This pull request removes some Japanese and Western language fonts that break emoji composition. I've also removed Lato from the New Toot modal for the same reasons.
Additionally, I've added Bootstrap's font stack as a fallback.
Fixes #708.

@@ -74,7 +74,7 @@ html, body, #app {
background-color: #d9e1e8;
}

--specified-fonts: 'Noto Sans', 'Noto Sans CJK JP', 'Takaoゴシック', 'Hiragino Kaku Gothic ProN', 'ヒラギノ角ゴ ProN W3', Meiryo, メイリオ, Osaka, 'MS PGothic', arial, helvetica, sans-serif;
--specified-fonts: 'Noto Sans', 'Noto Sans CJK JP', 'Takaoゴシック', 'ヒラギノ角ゴ ProN W3', '-apple-system', 'BlinkMacSystemFont', 'Segoe UI', 'Roboto', 'Helvetica Neue', 'Apple Color Emoji', 'Segoe UI Emoji','Segoe UI Symbol';
Copy link
Owner

Choose a reason for hiding this comment

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

[stylelint] posted by Sider

Expected single space after "," in a single-line list (value-list-comma-space-after)

@@ -319,7 +319,6 @@ export default {
resize: none;
height: 120px;
transition: border-color 0.2s cubic-bezier(0.645, 0.045, 9.355, 1);
font-family: 'Lato', sans-serif;
Copy link
Owner

Choose a reason for hiding this comment

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

Hmm... The reason of Lato is a bug of Noto Sans.
Noto Sans could not render backticks properly, so I have set Lato.
#382

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Found the issue, but I can't reproduce it. My Noto Sans version is 2.0 (courtesy of TeX Live). Perhaps it's been updated?

Copy link
Owner

Choose a reason for hiding this comment

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

I still reproduce it on my computer.
screenshot from 2018-11-21 21-40-43

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I can reproduce it too only with the embedded version in Whalebird.

Noto Sans 2.x works correctly.

Copy link
Owner

Choose a reason for hiding this comment

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

Oh, I see, thank you.
I'll update Noto Sans in Whalebird.

@h3poteto h3poteto merged commit d2c8c14 into h3poteto:master Nov 21, 2018
@amyspark amyspark deleted the fix-fonts branch November 21, 2018 15:32
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.

None yet

2 participants