Skip to content

Scaladoc: Replace UTF8 glyphs with the Material Typeface #4973

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

Merged
merged 1 commit into from
Feb 17, 2016

Conversation

felixmulder
Copy link
Contributor

This commit removes arrows and other symbols previously provided by
UTF8-symbols. Instead it uses the Material Typeface that we've used
for the search icon and graph symbols.

review: @VladUreche, @lrytz

I'm pulling Lukas into this one since he's the one whose phone was affected by this bug.

Docs for this PR available at: https://dl.dropboxusercontent.com/u/358427/scaladoc-glyphs/scala/Boolean.html

@scala-jenkins scala-jenkins added this to the 2.12.0-M4 milestone Feb 17, 2016
@felixmulder felixmulder changed the title Replace UTF8 glyphs with the Material Typeface Scaladoc: Replace UTF8 glyphs with the Material Typeface Feb 17, 2016
@VladUreche
Copy link
Contributor

@heathermiller may be in a better position to comment on this patch.

@lrytz
Copy link
Member

lrytz commented Feb 17, 2016

LGTM!

This commit removes arrows and other symbols previously provided by
UTF8-symbols. Instead it uses the Material Typeface that we've used
for the search icon and graph symbols.

review: @VladUreche, @lrytz
@felixmulder felixmulder force-pushed the topic/scaladoc-replace-utf8-glyphs branch from 046bd5c to 42ae388 Compare February 17, 2016 15:00
@heathermiller
Copy link
Member

Nice :) LGTM!

lrytz added a commit that referenced this pull request Feb 17, 2016
…-glyphs

Scaladoc: Replace UTF8 glyphs with the Material Typeface
@lrytz lrytz merged commit 1564f28 into scala:2.12.x Feb 17, 2016
@adriaanm adriaanm added 2.12 and removed 2.12 labels Oct 29, 2016
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.

6 participants