Skip to content

Conversation

@marcobeltempo
Copy link

Fixes mozilla/thimble.mozilla.org#2554

  • Added Courier new to remaining uses of courier.

Read this issue for more info
mozilla/thimble.mozilla.org#2552

@humphd humphd requested a review from flukeout November 27, 2017 16:22
@humphd
Copy link

humphd commented Nov 27, 2017

Adding @flukeout for review on this.

@flukeout
Copy link

flukeout commented Dec 1, 2017

Awsome, thank you! Left a small comment about indentation in one of the stylesheets.

}

#PDFBug .stats {
font-family: "Courier New", Courier, monospace !important;
Copy link

Choose a reason for hiding this comment

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

Small nit, could you please indent this line to match the rest of the stylesheet?

Copy link
Author

Choose a reason for hiding this comment

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

Ahhh good eye @flukeout Just fixed it

@humphd humphd requested a review from flukeout December 4, 2017 18:43
@humphd
Copy link

humphd commented Dec 4, 2017

Re-flagging @flukeout for a followup review.

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