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

lab-site: Add left panel for notes #403

Merged
merged 1 commit into from
Aug 5, 2024
Merged

lab-site: Add left panel for notes #403

merged 1 commit into from
Aug 5, 2024

Conversation

juliaogris
Copy link
Member

Add left panel for notes added as .htmlf HTML file fragment. A sample x.evy with
property "notes": true in samples.json will load x.htmlf and present the
contents in the left side panel on lab.evy.dev. Rework examples to include
notes. Generate SVG files from evy source code.

I considered breaking the commits up a bit further, but decided against it
because they all belong to the same: side notes for lab 1 issue.

Copy link
Member

@camh- camh- left a comment

Choose a reason for hiding this comment

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

I haven't completed reviewing this yet because I'm stuck again knowing what is generated and what isn't. This PR should have an update to .gitattributes to mark the generated files in /frontend/labs

I have some comments on the notes themselves but I'm not sure where to make those comments

Makefile Outdated Show resolved Hide resolved
Base automatically changed from labsite-notes-prep to main August 5, 2024 02:11
Add left panel for notes added as .htmlf HTML file fragment. A sample x.evy with
property `"notes": true` in samples.json will load x.htmlf and present the
contents in the left side panel. Rework examples to include notes. Generate SVG
files from evy source code.
Copy link
Member

@camh- camh- left a comment

Choose a reason for hiding this comment

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

📗

@juliaogris juliaogris merged commit 81374a9 into main Aug 5, 2024
3 checks passed
@juliaogris juliaogris deleted the labsite-notes branch August 5, 2024 02:15
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