Add option to toggle ruler visibility

[zeroishero]

Closes #1416
I added the menu, and made it send message to frontend to conditionally render the rulers element in svelte. I am using UpdateDocumentRuler and sending visible which is updated to rulersVisible. It works as intended but the markings and text is not being displayed. Here, as far as I know, the values necessary for computing markings and text is sent using UpdateDocumentRuler, and it has the data. I also tried updating spaces and interval and it did update it.

I watched the contribution video as guided by @Keavon and the process does match. I had tried making ToggleRuler message but it didn't send the data related to rulers along with it.

[0HyperCube]

@zeroishero the issue is that the svg for the numbers is set with width=0 and height=0. This can be fixed by adding onMount(resize); to RulerInput.svelte. You may need to add import { onMount } from "svelte"; at the top of the js script.

[0HyperCube]

Note that the tests failed to because of the new rulers_visible field prevents the document for deserialising. To fix this you can simply set serde to use a default value of true - see serde-rs/serde#1030 (comment).

[0HyperCube]

Works well, thanks for this contribution.

[Keavon]

I just completed my code review and pushed some improvements split out into several commits. So you can learn from the changes I made, I'd encourage you to read over each of the commits. One other note for future consideration: when submitting PRs, please do so from a branch in your fork other than master so we can check out your branch without its name conflicting with our copy of master, and to make it easier for you to keep your local fork up to date with our master branch (since there will be conflicting history when your work is based on master). Thanks for this feature, congrats on your contribution!

[Keavon]

!build

[github-actions]

📦 Build Complete for 4cddea5
https://ad8a61c6.graphite.pages.dev

[zeroishero]

I was waiting for it to be reviewed and planned to make recommended changes myself. Anyway, thanks. I will look at changes.

[Keavon]

Yeah, sometimes we do code review by leaving comments, other times it's just easier to fix the things directly and push them. Having you fix the comments is better for your learning but more effort to describe on my part, so I took the more efficient route this time. I hope you can learn from the commit message + implementation to see. I also followed this up with one more commit which fixed a bug that I introduced (but didn't notice until after merging), so consider this change to be a part of the things you look at to learn through osmosis: 9224ed9

Thanks again!

[Keavon]

Hi @zeroishero, thanks again for this code contribution to the project! We're still hoping you will respond to the request to relicense this code. Please see #4208 ASAP, thank you!