Skip to content

expand text below graphs; uniformise break sizes - #61

Open
jaxonL wants to merge 1 commit into
gh-pagesfrom
jl/text-reformat
Open

expand text below graphs; uniformise break sizes#61
jaxonL wants to merge 1 commit into
gh-pagesfrom
jl/text-reformat

Conversation

@jaxonL

@jaxonL jaxonL commented Jul 4, 2021

Copy link
Copy Markdown

i'm opening these changes in a separate PR since they are a point of contention (and i'd like #59 to go in earlier).

changes

  • uniformise breaks between sections, paragraphs (vbs, vbx2 instead of vbx3 or vbx4)

  • let text go below graphs (see examples below)

    before:

    image

    image

    after:

    image

    image

    (vertically, the text looks like this:)

    image

@jaxonL

jaxonL commented Jul 4, 2021

Copy link
Copy Markdown
Author

@AkshayPall has good points though:

Not to diminish all this work but I personally dislike having the 2 columns text - it should be clear that text is always on the left to read and graphs on the second (ignoring mobile / narrow displays ofc).

Also, when the text spills across columns, it looks overwhelming and chaotic to me when surrounded by left exclusive or dual column text. Idk what the others' thoughts are on this though :/

Maybe there are some papers we can consult related to natural reading flow.

i don't know if it seems messier (it may), though it does reduce the white space we have below graphs that accompany long text

i hope that responds a bit as to why i did this :x

@jaxonL
jaxonL force-pushed the jl/text-reformat branch from f7f657f to a644f3d Compare July 4, 2021 04:05
@netlify

netlify Bot commented Jul 4, 2021

Copy link
Copy Markdown

✔️ Deploy Preview for frosty-lichterman-99c65e ready!

🔨 Explore the source changes: f7f657f

🔍 Inspect the deploy log: https://app.netlify.com/sites/frosty-lichterman-99c65e/deploys/60e1328a633fd50007945275

😎 Browse the preview: https://deploy-preview-61--frosty-lichterman-99c65e.netlify.app

@netlify

netlify Bot commented Jul 4, 2021

Copy link
Copy Markdown

✔️ Deploy Preview for frosty-lichterman-99c65e ready!

🔨 Explore the source changes: a644f3d

🔍 Inspect the deploy log: https://app.netlify.com/sites/frosty-lichterman-99c65e/deploys/60e13394e057550007ca5182

😎 Browse the preview: https://deploy-preview-61--frosty-lichterman-99c65e.netlify.app

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.

1 participant