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

Add Static HTML Renderer and Documentation #204

Merged
merged 14 commits into from Aug 1, 2020
Merged

Conversation

carson-katri
Copy link
Member

@carson-katri carson-katri commented Jul 22, 2020

I wrote this while documenting the process of building a Renderer. I've attached the markdown files, which could be included in the Wiki.

The renderer relies on TokamakDOM so I didn't have to rewrite the HTML/CSS representations. It currently doesn't support App/Scene, but will once that gets merged. I'll also add in how to support that to the Wiki articles.

RendererWiki.zip

There's also a new demo TokamakStaticDemo. You can test the renderer with swift run TokamakStaticDemo.

Todo:

  • Support App and Scene
  • Swap dependencies so TokamakDOM depends on TokamakStatic
  • Rename to TokamakStaticHTML

@carson-katri carson-katri added documentation Technical writing, references, tutorials etc enhancement New feature or request labels Jul 22, 2020
@carson-katri carson-katri changed the title Add Static HTML Renderer Add Static HTML Renderer and Documentation Jul 22, 2020
@j-f1
Copy link
Member

j-f1 commented Jul 22, 2020

Maybe put the docs in the docs directory?

Package.swift Outdated Show resolved Hide resolved
Package.swift Outdated Show resolved Hide resolved
@carson-katri carson-katri marked this pull request as draft July 22, 2020 20:57
@MaxDesiatov
Copy link
Collaborator

Would you consider adding redacted as a separate PR to make this one smaller and easier to review?

@carson-katri
Copy link
Member Author

Sure no problem.

@carson-katri carson-katri marked this pull request as ready for review August 1, 2020 03:02
MaxDesiatov
MaxDesiatov previously approved these changes Aug 1, 2020
Copy link
Collaborator

@MaxDesiatov MaxDesiatov left a comment

Choose a reason for hiding this comment

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

I didn't test it locally yet, but this is amazing and I can't wait to start building something with it 👏

MaxDesiatov
MaxDesiatov previously approved these changes Aug 1, 2020
j-f1
j-f1 previously approved these changes Aug 1, 2020
Sources/TokamakDOM/Views/DynamicHTML.swift Outdated Show resolved Hide resolved
@MaxDesiatov
Copy link
Collaborator

Sorry about the conflicts, I'll get those fixed

# Conflicts:
#	Sources/TokamakDOM/DOMRenderer.swift
#	Sources/TokamakDOM/Views/Buttons/Button.swift
@MaxDesiatov MaxDesiatov added the API design API design and prototyping is needed label Aug 1, 2020
@carson-katri carson-katri merged commit 4c654da into main Aug 1, 2020
@carson-katri carson-katri deleted the static-renderer branch August 1, 2020 20:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API design API design and prototyping is needed documentation Technical writing, references, tutorials etc enhancement New feature or request
Development

Successfully merging this pull request may close these issues.

None yet

3 participants