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

Graphics App: adding a setting to hide or display the app in the tablet/toolbar #1275

Merged
merged 1 commit into from
Dec 20, 2024

Conversation

AleziaKurdis
Copy link
Contributor

This make the tablet app to be launched or not based on new setting that control its present in the tablet/toolbar. It also make it able to discard or restore itself on the tablet/toolbar if the option is checked in the Setting menu. (See PR 1274 for the new setting)

By default the app will be visible.

TESTING:
This app must be tested once the PR 1274 will be present in the code.
(It hasn't been tested yet)

  • Test if the app is visible and functional if the setting is find to true at startup.
  • Test if the app is not visible if the setting is find to false at startup.
  • Test if the app get visible and functional if the setting is changed to true at runtime.
  • Test if the app get not visible if the setting is changed to false at runtime.

This make the tablet app to be launched or not based on new setting that control its present in the tablet/toolbar.
It also make it able to discard or restore itself on the tablet/toolbar if the option is checked in the Setting menu.
(See PR 1274 for the new setting)
@ksuprynowicz ksuprynowicz added CR approved This pull request has been successfully code reviewed QA approved This pull request has been successfully tested labels Dec 20, 2024
@ksuprynowicz
Copy link
Member

I just tested it, works perfectly :)

@ksuprynowicz ksuprynowicz merged commit fba519e into overte-org:master Dec 20, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CR approved This pull request has been successfully code reviewed QA approved This pull request has been successfully tested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants