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

CSS improvements to github-pandoc.css #159

Merged
merged 5 commits into from
Jan 22, 2019
Merged

CSS improvements to github-pandoc.css #159

merged 5 commits into from
Jan 22, 2019

Conversation

vincerubinetti
Copy link
Collaborator

  • fix superscripts per greenelab/meta-review#88 (comment)
  • fix printing page margin css to be even 0.75in
  • hide hypothesis sidebar and tooltip on print
  • fixed style from pandoc that made long/wide code boxes overflow past their container and look bad

@dhimmel dhimmel changed the title Css fixes CSS improvements to github-pandoc.css Jan 19, 2019
Copy link
Member

@dhimmel dhimmel left a comment

Choose a reason for hiding this comment

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

Nice. Can you upload the PDF created by build.sh for the meta-review with this CSS file?

margin-left: 0.75in;
margin-right: 0.75in;
}
/* Remove margin, padding, border, etc. when printing; go right up to @page pargins. */
Copy link
Member

Choose a reason for hiding this comment

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

typo pargins

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed

@@ -262,7 +275,7 @@ sup {
}

sup {
top: -0.5em;
top: -0.1em;
Copy link
Member

Choose a reason for hiding this comment

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

Will this fix the superscript appearance issues described at greenelab/meta-review#88 (comment)?

Copy link
Collaborator Author

@vincerubinetti vincerubinetti Jan 22, 2019

Choose a reason for hiding this comment

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

No, that's just to make it look a little better (in my judgement). The real culprit of the problem was this:

ba17b84#diff-f52f320f93db27d62530fdd59fbcfb06L789

@@ -289,6 +302,25 @@ svg:not(:root) {
overflow: hidden;
}

/**
* Hide hypothesis annotation toolbar, tooltip, etc. on print.
*/
Copy link
Member

Choose a reason for hiding this comment

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

Nice. It was annoying that printing would show the hypothesis toolbar. Does it also remove hypothesis highlights?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

No, I figured a user might want to retain those when printing? We should be easily able to remove that if you want though.

Copy link
Member

Choose a reason for hiding this comment

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

No, I figured a user might want to retain those when printing?

Agreed

@dhimmel
Copy link
Member

dhimmel commented Jan 22, 2019

Can you upload the PDF created by build.sh for the meta-review with this CSS file?

PR looks good to go pending review of the WeasyPrint PDF export. Let me know if you need help setting up the conda environment so sh build/build.sh runs properly.

@vincerubinetti
Copy link
Collaborator Author

manuscript.pdf

@dhimmel dhimmel merged commit eb4cb3e into manubot:master Jan 22, 2019
dhimmel pushed a commit that referenced this pull request Jan 22, 2019
This build is based on
eb4cb3e.

This commit was created by the following Travis CI build and job:
https://travis-ci.org/greenelab/manubot-rootstock/builds/483107422
https://travis-ci.org/greenelab/manubot-rootstock/jobs/483107423

[ci skip]

The full commit message that triggered this build is copied below:

CSS improvements to github-pandoc.css

Merges #159

When printing, hide hypothesis toolbar and fix page margins.

Fix pandoc syntax highlighting quirk: long width code boxes
overflow outside of container.

Fix superscript size issue documented at
greenelab/meta-review#88 (comment)
dhimmel pushed a commit that referenced this pull request Jan 22, 2019
This build is based on
eb4cb3e.

This commit was created by the following Travis CI build and job:
https://travis-ci.org/greenelab/manubot-rootstock/builds/483107422
https://travis-ci.org/greenelab/manubot-rootstock/jobs/483107423

[ci skip]

The full commit message that triggered this build is copied below:

CSS improvements to github-pandoc.css

Merges #159

When printing, hide hypothesis toolbar and fix page margins.

Fix pandoc syntax highlighting quirk: long width code boxes
overflow outside of container.

Fix superscript size issue documented at
greenelab/meta-review#88 (comment)
@vincerubinetti vincerubinetti deleted the css-fixes branch March 12, 2019 13:45
adebali pushed a commit to CompGenomeLab/lemur-manuscript-archive that referenced this pull request Mar 4, 2020
Merges manubot/rootstock#159

When printing, hide hypothesis toolbar and fix page margins.

Fix pandoc syntax highlighting quirk: long width code boxes
overflow outside of container.

Fix superscript size issue documented at
greenelab/meta-review#88 (comment)
ploegieku added a commit to ploegieku/2023-functional-homology-paper that referenced this pull request Aug 6, 2024
Merges manubot/rootstock#159

When printing, hide hypothesis toolbar and fix page margins.

Fix pandoc syntax highlighting quirk: long width code boxes
overflow outside of container.

Fix superscript size issue documented at
greenelab/meta-review#88 (comment)
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.

2 participants