-
Notifications
You must be signed in to change notification settings - Fork 76
feat: Add documentation for slint and flutter #608
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
base: master
Are you sure you want to change the base?
Conversation
|
New warnings found with rstcheck: |
| .. _GUI_Frameworks_User_Guide: | ||
|
|
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.
This label is unnecessary right now.
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.
I have added this reference in source/linux/Demo_User_Guides/TI_Apps_Launcher_User_Guide.rst file.
| echo "meta-slint,https://github.com/slint-ui/meta-slint.git,main,7406ee51c140968345b86f3927a8c67984a2dda8,layers=" >> configs/slint-config.txt | ||
| echo "meta-rust-bin,https://github.com/rust-embedded/meta-rust-bin.git,master,79c077fac9694eb5fbcee7b15e800c21e887bb5d,layers=" >> configs/slint-config.txt |
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.
Does this need to be documented? This is a standard guide on how to integrate a new layer into an existing config. The hash will almost immediately be invalid for the end user, unless you are prepared to update it every release...
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.
These are the hashes that are tested for this release. So, the end users has the working hash that they can start with.
From next release these are maintained in the oeconfig files directly and the instructions will be edited similar to flutter instructions (line number 45)
888d746 to
09557ac
Compare
|
New warnings found with rstcheck: |
Add a how to develop guide with different UI frameworks including slint and flutter in the Demo Applications section for AM62x/AM62P. Signed-off-by: Telukula Jeevan Kumar Sahu <j-sahu@ti.com>
09557ac to
ffcff3d
Compare
|
New warnings found with rstcheck: |
| .. note:: | ||
|
|
||
| The TI Apps Launcher uses the Qt framework. The SDK also supports other modern UI toolkits, | ||
| such as Flutter and Slint. For instructions on building applications with these frameworks, | ||
| see the :ref:`GUI_Frameworks_User_Guide`. | ||
|
|
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.
Why a note? This is just a general statement. Not really noteworthy in my opinion.
| ############# | ||
| UI Frameworks | ||
| ############# |
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.
Mixed usage of GUI and UI in this document. This is solely about graphical user interfaces.
| See the :ref:`TI-Apps-Launcher-User-Guide-label` documentation for more information about Qt-based demos. | ||
| This guide provides a brief overview of Flutter and Slint, along with instructions on how to build and run demos for each. |
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.
Should we really be directing people here? We already have a Qt overview and listed demos under the graphics section.
| .. code-block:: bash | ||
| systemctl stop ti-apps-launcher | ||
| WAYLAND_DISPLAY=/run/user/1000/wayland-1 LD_LIBRARY_PATH=/usr/share/flutter/flutter-samples-material-3-demo/3.38.3/release/lib flutter-client --bundle=/usr/share/flutter/flutter-samples-material-3-demo/3.38.3/release |
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.
You should not tell people to run graphical applications as root.
Add a how to develop guide with different UI frameworks including slint and flutter in the Demo Applications section for AM62x/AM62P.