Development Issues
Issues faced during development of pre-prints
PagedJs Issues:
When we do manual changes, pagedJS no longer adjusts the margin of the :first-element (for the specific element only)
J: i don’t understand completely. the firs-element thing was used to remove the white space on top of a title, we shouldn’t use it now, as it bring too much trouble.
H: Okay!
heading + table with multiple images still breaks. PagedJs adds an empty image after heading and adds content to next page
break-before: always
. If this is written in :is() {break-before: always}, pagedJS throws error. In regular cases, it works sometimes
empty figure with id on one page, actually figure with data-id on the next page. To access the figure and make changes, I’ve to use J: data-id are used by pagedjs so we shouldn’t have any data-id in our css. the problem comes from the fact that we try to keep the caption and the image together.
H: Yep, but when there are multiple images inside one figure and pagedJS breaks the across pages, each of those get wrapped by a figure that has[data-id]. And, there is no other way to style them.
Code Issues:
prettified link must only be for required element and not all links
J: to fix that, we can check if the href for the link is actually the same. (basically add a script that check if the link is the same, without looking at https)
H: don't get you
break-after: avoid
. It leaves blank spaces when we have multiple block-quotes together or if block-quote content is big.
block-quotes shouldn't have J: let’s remove that
H:
multiple headings + content leave orphaned headings behind
J: I made a script for that. is it not working?
H: It does work most of the time, Check css in #25 (closed) for examples. Lookout for selectors starting with #s or #el-h1 or #el-h2
heading + p + image, separates images from heading (or heading + link + img)
data-id doesn't work sometimes (might be pagedJS issue)
J: never use data-id
H: refer top for reply
need to add separate styling for images with only heading in figcaption
J: can we add a script to pagedjs to check if the image contains more than a title?
H: yes
some images(portrait) are sleek, need to have a separate style for them
J: unclear. could we rephrase this?
H: The images are long, their width is only 100px while height goes up to 450px. So, they go completely out of the main content area and the figcaption goes across the whole content space. This is okay when there is enough content in caption. When there is only a small para, then there is a huge white space in the main content area.
[new] For, heading + list, heading and one bullet is on one page and the rest of the bullets on another, is that okay? (Manually fixed it for latest batch)
Content Issues:
Bold para for Reviewer heading, Author Response, References, All Author response subheadings
J: Those need to be changed in the html to have the right behavior. is that right?
H: yes, most of these changes need to come from the HTML
block-quotes need to come together
J: not clear what you mean
H: Currently, heading and content are being wrapped in separate blockquotes. We discussed this earlier. Added it here to keep track of what's pending
class names aren't consistent
J: do you have examples for that?
H: yeah, supplementary figures can be figs1 or figS1 or fig1_s1. Also, earlier they were according to the caption title, in the past batch they were sequential
for tables, the heading row should never be a separate image,pre-print => page 1[ heading + heading row image + blank spaces] page2 [rest of the table]
J: wait what? we have images for heading row? where can i find this example?
H: 87458.1 table S7
a char is 50px sometimes 16px
formula are not properly sized. For example, sometimes J: that’s because they are images. there is no simple way to fix this, it need to be accepted as such, or manual work.
content images are sized beyond what's needed, if this continues, will need to keep styling them manually
J: I have a script to force the size on the image so they are in relation: the biggest image will be considered at 100% and all the other will be reduced based on the ratio we use from the image size to the pdf output size.
H: Won't that shrink some of the perfectly sized images, or make an image that's already small tiny?
images should not have blank spaces around the content, in the PDF it'll look like additional margins
J: we can’t remove that
H: So, we manually fix the margins for those always?
How are we supposed to handle author name when names are to long to fit in the runninghead(did elipsis last time)(if that's okay, we can add it globally for all files)
J: use ellipsis. it’s a running head, not even sure we need the author names in there.
H:
images need to be a reasonable size, when there is too much content in 1 image, we can't understand the content even after resizing it to all the available space
J: we can’t do anything about that. since there is no production team in the author world.
H: then, it makes no sense having that image
Bullet points markers are in the HTML. And, the marker and content are wrapped in separate paragraphs. This shows 2 markers, and the content and markers are on separate lines.
J: that’s up to elife to fix
Abstract should have 1 heading only
J: do you hvae an example of one?
It isn't clear whether this is supposed to be content or heading
J: supposed to be headings. but it can’t be set that way.