Skip to content

Add two style tweaks to improve tables in code blocks #738

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

Merged
merged 1 commit into from
Jan 20, 2023

Conversation

waldyrious
Copy link
Contributor

Demo using the latest blog post (at the time of writing) as an example.

Before:
image

After:
image

CC @hustcer who did a lot of work in the Vuepress setup in #465.

Also CC @Aloso who did similar tweaks back in #8 + #9.

@fdncred
Copy link
Contributor

fdncred commented Jan 20, 2023

Let's try it out.

@fdncred fdncred merged commit 83ced1d into nushell:main Jan 20, 2023
@fdncred
Copy link
Contributor

fdncred commented Jan 20, 2023

A little better but not quite fixed using macos and msedge.
image

@waldyrious waldyrious deleted the patch-1 branch January 20, 2023 13:48
@waldyrious
Copy link
Contributor Author

A little better but not quite fixed using macos and msedge.

I can't test on those browsers, but if it helps, when preparing this PR I used Firefox's developer tools, which has a really nice Font panel that is able to show the individual characters using each font. I basically experimented with tweaking the CSS in my browser until it looked OK in my machine, and submitted that patch because it was a strict improvement over the previous state.

@fdncred
Copy link
Contributor

fdncred commented Jan 20, 2023

Looks good on Windows with msedge. I'm wondering if none of those fonts were found on macos?
image

@fdncred
Copy link
Contributor

fdncred commented Jan 20, 2023

I looked and it seems that the problem is that it's choosing Monaco on MacOS, which is a monospace font but not a great one for drawing pictures. If I change it to "FiraCode Nerd Font Mono" or "Iosevka Nerd Font Mono" or "CaskaydiaCove Nerd Font Mono" it looks great. I guess it's just about what fonts you have on your system and which one it finds first. Maybe I can find a Consolas for mac.

@fdncred
Copy link
Contributor

fdncred commented Jan 20, 2023

Consolas was a bust. Looked terrible on MacOS.

@hustcer
Copy link
Contributor

hustcer commented Jan 21, 2023

@waldyrious I'm terribly sorry for keep you waiting so long, I have just noticed this PR after it has been merged. Maybe too many notifications make me missed this one.

@waldyrious
Copy link
Contributor Author

No worries @hustcer, that's perfectly understandable! Just let me know what you think of the changes :)

@hustcer
Copy link
Contributor

hustcer commented Jan 23, 2023

@waldyrious It looks good to me, thanks

ayax79 pushed a commit to ayax79/nushell.github.io that referenced this pull request Jun 26, 2024
Co-authored-by: nash <nash@iffy.me>
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