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

Page about HashLink #104

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

marcelEuchnerMartinez
Copy link
Contributor

  • how to use hot reload
  • common libraries (also mentioned in installing section)

Basic instructions on how to use hot reload. use case/ limitations.
providing graphics and sounds for HashLink
link to the other libs available to HL
@hristo-kanchev
Copy link
Contributor

Nice!

This is just an idea that the content of the page "Development" may be put into the First-Outlook page.

---
That the development section exists at all is due to the history of Heaps and the work of Deepnight.

It would also be possible to hide this page instead somewhere, but at the moment we don't really have other options.
First-outlook.md Outdated
@@ -56,3 +56,13 @@ Heaps is powered by Haxe. Learn more about Haxe here:
- The [Haxe code cookbook](https://code.haxe.org/)
- The [Haxe community](https://community.haxe.org/)

## More on game engine development
Copy link
Contributor

@hristo-kanchev hristo-kanchev Oct 24, 2024

Choose a reason for hiding this comment

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

Can you delete Development.md and remove it from _Sidebar.md so we don't have duplicate information?

Also, maybe rename this header to "More on game development" instead of "More on game engine development" as the info you are sharing here doesn't directly relate to actual "engine" development.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

(I shifted this to a parallel pull request for the moment)

@yuxiaomao
Copy link
Contributor

I'm personally not very fan of these changes,
lib.haxe.org is only for versioned sdl/directx, not for hashlink git master, so the link can be misleading;
it maybe worth mentioning openal but is not an obligation for heaps to run - if the game do not have sound or would like to use other sound solution;
hot-reload is still experimental and is a duplication of hashlink wiki.

@marcelEuchnerMartinez
Copy link
Contributor Author

marcelEuchnerMartinez commented Oct 25, 2024

I removed the "HashLink" page entirely and mentioned the hot reload function in the "Hello-HashLink" page instead (with pointing out it is still under development, though!)

I could now also remove the pages "Target platforms" and "HTML5" ?
Because the information is mostly already in the "Hello-World" and "Hello-HashLink" pages.
(Also they are mentioned in "Home")

(Sidebar would also get smaller, though a collapsible sidebar would also still be welcome :)

@nayata
Copy link

nayata commented Oct 25, 2024

Removing "Target platforms" and "HTML5" pages is the very bad idea.

Its very not intuitive that information about, for example, html5 you should search in the Hello World page.

@marcelEuchnerMartinez
Copy link
Contributor Author

marcelEuchnerMartinez commented Oct 25, 2024

Okay, then we should keep all these pages. They can provide additional information maybe later on.

…ashlink page"

Keeping the HashLink page!

This reverts commit 38ba71e.
…ashLink: changing the link to lib.haxe.org instead the hashlink repo
@marcelEuchnerMartinez
Copy link
Contributor Author

Alright, then let's close this pull request.

I messed this up a little 🥴 (but Hristo will probably check the page at some point later instead)

(But in the long run HL hot reload is a very attractive feature for developers. Maybe it could/should be explained on the site of Heaps. Also the HashLink wiki is a bit more technical/abstract/general purpose... From the perspective of Heaps it can be explained more concretely what is can be used for.)

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.

4 participants