search

rokwire / illinois-app

Source code repository of "Illinois" App - the official mobile app of the University of Illinois.
https://app.illinois.edu/
Apache License 2.0
23 stars 20 forks source link

[FEATURE] New Assistant UI #4089

Closed dobromirdobrev closed 3 months ago

dobromirdobrev commented 3 months ago

Fwd: Illinois App • Vanessa added new screens / 2024/06/03

Could you take a look at this please and make the changes to the Assistant UI please

Ref: UI mockup

IllinoisAssistantUI-2024-05-31.pdf

dobromirdobrev commented 3 months ago

Hi @vburgett ,

We started to work on the new Assistant UI. So, a couple of questions arose in this regard.

  1. What should "Clear All" link button do?
  2. We need more precise .png files for the Assistant tab icon (please see the attached screenshots). The background color should be transparent (not white) and not to be cut off at the side. Current icons have white background and are cut off at top, right and bottom sides. Could you please provide them?
Screenshot 2024-06-03 at 14 28 52 Screenshot 2024-06-03 at 16 38 11

Other questions comments may appear during the development process. Thanks cc @pmarkhennessy

vburgett commented 3 months ago

@dobromirdobrev attached are new pngs files for the icon.

“Clear All” removes everything in the thread but the initial prompt from the Assistant. Screenshot 2024-06-03 at 8 56 48 AM

Icon-IllinoisAssistant.zip

dobromirdobrev commented 3 months ago

@vburgett ,

Thank you! Somehow I missed the "Clear All" behaviour description in the .pdf file.

Regarding the icons, the new ones still have white background. Please see the attached screenshot:

Screenshot_20240604-174050.png

The other tab icons have transparent background while the Assistant icon has white one. Maybe this is not so important but I just wanted to let you know what it would look like.

vburgett commented 3 months ago

Thank you for the screenshot, @dobromirdobrev. The assistant icon has a white outline around it with the block I filled in with white. That is intentional. It does look odd against yellow but it will look more cohesive when it is up against the white background.

dobromirdobrev commented 3 months ago

Hi @vburgett , This is implemented in version 6.0.5 of the Illinois app. You can verify it when you have a chance.

vburgett commented 3 months ago

Thank you, @dobromirdobrev.

During a session, the Illinois Assistant thread should not be cleared when going to another panel like the Illinois Assistant FAQs or Browse for example. The thread should be visible to the user during that session unless they "clear all."

dobromirdobrev commented 3 months ago

@vburgett , Your latest requirement is done in version 6.0.6 of the illinois app. Please verify when you have a chance

vburgett commented 3 months ago

Confirming that the below is working in 6.0.6 on dev (Android). Thank you. @dobromirdobrev. During a session, the Illinois Assistant thread should not be cleared when going to another panel like the Illinois Assistant FAQs or Browse for example. The thread should be visible to the user during that session unless they "clear all."