-
-
Notifications
You must be signed in to change notification settings - Fork 27
Update README #37
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
Update README #37
Conversation
|
Very nice work! I would add a section on how to compile and run the examples. |
|
@Eagle941 Thank you for looking at it, I updated the PR based on your feedback 🎉 I have no clue about building or running the other examples as they are targeting something very different from what I am trying to do. Another PR would have to improve on their build instructions in the future. |
|
@tracernz Thank you, is it mergable then? |
README.md
Outdated
| background_color=0,0,255 | ||
| htmlgauge00=WasmInstrument/WasmInstrument.html?wasm_module=gauge_logic.wasm&wasm_gauge=Demo, 0,0,1024,768 | ||
| ``` | ||
| <img width="700" alt="grafik" src="https://github.com/user-attachments/assets/babe0670-51bf-4a33-8fc6-5e282667febe" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you check in this image as well?
Instead of hosting them on GitHub, based on feedback in PR flybywiresim#37
|
@Gurgel100 Thanks for the review, I added the image to the repo locally. The main FBW aircraft repo had the images as a link but this also works :) |
|
I'd love to get this merged to have it off my TODO list. Any blockers? |
|
Thanks, looks good! |
In order to make the msfs-rs project more accessible to beginners, I created a README.