Skip to content

chore: Make apps more intuitive in use#1006

Merged
msluszniak merged 10 commits intomainfrom
@ms/make-apps-more-intuitive
Mar 27, 2026
Merged

chore: Make apps more intuitive in use#1006
msluszniak merged 10 commits intomainfrom
@ms/make-apps-more-intuitive

Conversation

@msluszniak
Copy link
Copy Markdown
Member

@msluszniak msluszniak commented Mar 24, 2026

Description

This PR adds:

  • In LLM app - Adds better phrasing in voice chat example.
  • In CV app - Blocks buttons when image not selected & add brief description of an example. Additionally, adds model selection list to classification example.
  • In CLIP embeddings app - Removes text embedding comparison from this example (since it already is in text embeddings example) and make app more intuitive in use (chosen image is displayed). Also adds model selection list to both text and image embeddings.

Introduces a breaking change?

  • Yes
  • No

Type of change

  • Bug fix (change which fixes an issue)
  • New feature (change which adds functionality)
  • Documentation update (improves or adds clarity to existing documentation)
  • Other (chores, tests, code style improvements etc.)

Tested on

  • iOS
  • Android

Testing instructions

Run the following apps:

  • LLM
  • Computer Vision
  • Clip embeddings

And check if described changes works correctly and does not introduce any problems.

Screenshots

Related issues

Closes #351

Checklist

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have updated the documentation accordingly
  • My changes generate no new warnings

Additional notes

@msluszniak msluszniak self-assigned this Mar 24, 2026
@msluszniak msluszniak added documentation Improvements or additions to documentation demo app labels Mar 24, 2026
@msluszniak msluszniak force-pushed the @ms/make-apps-more-intuitive branch from 4dba41d to 2b5e544 Compare March 25, 2026 10:17
@msluszniak msluszniak requested review from barhanc and chmjkb March 25, 2026 10:22
@msluszniak
Copy link
Copy Markdown
Member Author

Changes from previous PR introduced some weird jumps in llm demo apps on iOS, working on it.

@msluszniak msluszniak marked this pull request as draft March 25, 2026 12:14
@barhanc
Copy link
Copy Markdown
Contributor

barhanc commented Mar 25, 2026

In computer vision app using Android back button (or swipe from edge gesture) goes to Vision Camera screen instead of the screen selection menu.

Copy link
Copy Markdown
Contributor

@barhanc barhanc left a comment

Choose a reason for hiding this comment

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

There is a problem with not being able to scroll the model list on Android in LLM and CV apps.

@msluszniak
Copy link
Copy Markdown
Member Author

In computer vision app using Android back button (or swipe from edge gesture) goes to Vision Camera screen instead of the screen selection menu.

@barhanc you mean vision camera example here? I thing that other apps don't have back button.

@barhanc
Copy link
Copy Markdown
Contributor

barhanc commented Mar 25, 2026

In computer vision app using Android back button (or swipe from edge gesture) goes to Vision Camera screen instead of the screen selection menu.

@barhanc you mean vision camera example here? I thing that other apps don't have back button.

No, I mean using the system navigation bar on Android.

@barhanc barhanc self-assigned this Mar 26, 2026
@barhanc barhanc marked this pull request as ready for review March 26, 2026 15:26
@msluszniak
Copy link
Copy Markdown
Member Author

I cannot approve my own PR (but you can add approve ;p), but every comment from you now works perfectly fine, so it's imo ready to ship it. 🚀

@barhanc barhanc self-requested a review March 27, 2026 12:25
@msluszniak msluszniak merged commit 3fe7c12 into main Mar 27, 2026
4 checks passed
@msluszniak msluszniak deleted the @ms/make-apps-more-intuitive branch March 27, 2026 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

demo app documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make using demo apps more intuitive

2 participants