PhenoApps / Field-Book

https://fieldbook.phenoapps.org
GNU General Public License v2.0
52 stars 54 forks source link

App intro #992

Closed kamathprasad9 closed 3 months ago

kamathprasad9 commented 4 months ago

Description

Adds app intro in experimental section of the app. Closes #864

Type of change

What type of changes does your code introduce? Put an x in boxes that apply.

Checklist:

Changelog entry

Please add a one-line changelog entry below. This will be copied to the changelog file during the release process.

Added app into

trife commented 4 months ago
chaneylc commented 4 months ago

Good start, I think this will be a great addition in replacement of all the start-up dialogs.

Some opinions of mine:

The welcome page should have a more succinct description.

The second page needs better abstract visuals that show the collection process, the title can probably also just be "Features" or "Phenotyping"

I think the permissions page is cool, but I think it would be better if the permissions automatically pop-up, requiring one less interaction from the user. The define storage description should be something else like: "Define a folder that will contain all field data."

Optional page has inconsistent titles. I would just say Load Sample Data / Tutorial instead of full questions. The Tutorial description should say something like "Enables visual indicators with textual descriptions of features throughout the app."

I think it would be good to add other optional features like @trife originally posted in https://github.com/PhenoApps/Field-Book/issues/864

It would be great if this could be disabled/automated on debug builds.

chaneylc commented 4 months ago
  • [x] Use main_primary_transparent for all slide backgrounds
    • [x] Set both indicator color and arrow color to main_primary_dark

These are specific to the default theme. We want to use the fb_... attributes if we plan to switch the theme for palma devices before the tutorial.

trife commented 3 months ago