Skip to content

Awkward unrelated comment alignment #2996

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

Closed
jdm opened this issue Sep 5, 2018 · 1 comment · Fixed by #3833
Closed

Awkward unrelated comment alignment #2996

jdm opened this issue Sep 5, 2018 · 1 comment · Fixed by #3833

Comments

@jdm
Copy link
Contributor

jdm commented Sep 5, 2018

Before:

        let normalized = self.ctfont.all_traits().normalized_weight();  // [-1.0, 1.0]
        // TODO(emilio): It may make sense to make this range [.01, 10.0], to
        // align with css-fonts-4's range of [1, 1000].

After:

        let normalized = self.ctfont.all_traits().normalized_weight(); // [-1.0, 1.0]
                                                                       // TODO(emilio): It may make sense to make this range [.01, 10.0], to
                                                                       // align with css-fonts-4's range of [1, 1000].
@scampi
Copy link
Contributor

scampi commented May 8, 2019

Add example from #3431 as test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants