derkork / godot-statecharts

A state charts extension for Godot 4
MIT License
766 stars 40 forks source link

Move the manual to Github Pages #146

Closed joelgomes1994 closed 3 days ago

joelgomes1994 commented 2 weeks ago

Hello, @derkork! Are you interested into moving the manual to a dedicated site (https://derkork.github.io/godot-statecharts) using Github Pages?

I made this site some time ago to document a tool I made, and I could use the same template, change the theme, logos and separate topics into different .md files. The hosting is using Github Pages (so no domains and hosting costs are required), and editing it is pretty straightforward (as the source documents are also Markdown). The source code is hosted in a separate branch, and deployed with Github Actions every time a change is pushed to the repository.

I could have made this and submitted a pull request already, but since I don't know if you're interested I thought it was better to ask first.

derkork commented 2 weeks ago

Hi @joelgomes1994! Well it sure looks nicer than the abomination we have right now - so it would definitely be an improvement (we could even have consistent image sizes, yay!). I'm not sure about the separate branch, this sounds a bit like merge conflict hell, maybe we could just use a tag to mark the published version?

joelgomes1994 commented 2 weeks ago

It's possible to deploy from a docs folder from the main branch:

image

The only problem is, when downloading the plugin from the Asset Library, it will also download the website source code as well (it's already doing that anyways). About the merge conflicts, the gh-pages branch is not even supposed to be merged from or to the main branch anyways, it would be totally dedicated to the docs, so I don't think it should be a concern.

But if you feel more comfortable, it's totally fine to use the docs folder in the main branch.

derkork commented 2 weeks ago

You can actually control what goes into the asset library zip, e.g. right now the manual is also part of the repo and is not inside the zip for the asset library (or if you download the zip manually from github). This is controlled through the .gitattributes file. So I'd prefer keeping this structure.

joelgomes1994 commented 2 weeks ago

I'm working on this task in a fork, the temporary site was deployed to https://joelgomes1994.github.io/godot-statecharts. For now, I just copied and pasted the manual contents into different pages and fixed image paths.

The chages are in the https://github.com/joelgomes1994/godot-statecharts/tree/gh-pages branch, but in the docs folder, so this could be easily merged into the main branch once the pull request is created.

It's using the following configuration: image

I'll fix image sizing and other CSS issues.

derkork commented 2 weeks ago

This looks very nice! Thanks a lot for taking the time to do this!

joelgomes1994 commented 1 week ago

Submitted the pull request #147

derkork commented 1 week ago

This is awesome thank you very much! I'll take this opportunity and slightly rework the manual so I will make some local changes on the branch before merging it. Again a huge thanks for doing this!

derkork commented 3 days ago

And merged! A big thanks again! I have taken the liberty to add your name to the footer, this would not have been possible without your work so I feel you deserve credit for this. Please let me know if this isn't ok with you, then I will remove it.

joelgomes1994 commented 3 days ago

And merged! A big thanks again! I have taken the liberty to add your name to the footer, this would not have been possible without your work so I feel you deserve credit for this. Please let me know if this isn't ok with you, then I will remove it.

That's great! I would prefer a link to my profile such as:

Website created by <a target="_blank" href="https://github.com/joelgomes1994" style="color: #e18e39; text-decoration: none;">joelgomes1994</a>

image

Takes less space and is more useful. :smiley_cat:

And thanks for merging the PR, nice work maintaining this project!

derkork commented 3 days ago

Done! Thanks again!