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

UI/UX changes for showcase #1402

Closed
mahalakshme opened this issue May 15, 2024 · 7 comments
Closed

UI/UX changes for showcase #1402

mahalakshme opened this issue May 15, 2024 · 7 comments
Assignees

Comments

@mahalakshme
Copy link
Contributor

Issues:
https://drive.google.com/file/d/1Qu-jvYeHgBpsaqOGesa_4Jxb87k_oUT8/view?usp=sharing - Video that has issues pointed out

  • Program enrolment audit details need to display in the Enrolment details section not in the Summary section
  • Need to change the format(Enrolled on 12 Nov 2023 by Suresh) of audit as in the mockups. Same for registeration details in Profile tab as well.
  • On clicking edit of Enrolment details, top bar that has the name is white in color and hence not visible - change the name to black
  • Summary table of Enrolment looks separate, it also needs to have grey background like we have for encounter table so that it looks as one unit
  • When the beneficiary has not enrolled in any program we show some gray bg unnecessarily
  • There is some grey footer unnecessarily
  • Relative box seems to be too bigger, move the DELETE button somewhat closer to other details
  • When clicking on Menu -> dashboard, the filter icon should not be in the top right since the filters are not applicable for all dashboards, Hence it needs to display below the horizontal list of dashboards we display and above the selected filters.
  • The selected filters are displayed with grey bg in the mockup. Currently we have blue bg, so the same can be done if it looks good.
  • Currently there are different kinds of buttons which is somewhat looking wierd. So make all primary buttons like Do(on scheduled visits) similar to the one in mockups(green button with white text). Same for Edit, Enrol buttons as well
  • Also on click of button we are showing grey border which also looks wierd. So, just replace it with ripple effect.
@mahalakshme mahalakshme converted this from a draft issue May 15, 2024
@mahalakshme mahalakshme changed the title sample UI/UX changes for showcase May 15, 2024
@mahalakshme mahalakshme moved this from In Analysis to Ready in Avni Product May 15, 2024
@petmongrels petmongrels moved this from Ready to In Progress in Avni Product May 16, 2024
@petmongrels petmongrels self-assigned this May 16, 2024
@petmongrels
Copy link
Contributor

Please verify colors in the app during code review review and QA.
For ripple the buttons are too small and transition too fast to see the ripple, but it looks better than displaying the border only like it was earlier.

@petmongrels petmongrels moved this from In Progress to Code Review Ready in Avni Product May 16, 2024
@himeshr
Copy link
Contributor

himeshr commented May 16, 2024

Have reviewed the code changes on github, looks good.

@mahalakshme mahalakshme moved this from Code Review Ready to In Code Review in Avni Product May 17, 2024
@mahalakshme mahalakshme moved this from In Code Review to In QA in Avni Product May 17, 2024
@mahalakshme
Copy link
Contributor Author

It looks good. But found some minor improvements that can be made:
https://github.com/avniproject/avni-client/assets/10369963/28aa8414-ad7d-430c-9854-e44a0f49820e

  • 'Enrolment details' and 'Visits planned' sections are not separated
  • Secondary buttons like Exit, cancel we were displaying in red text before - it is good to do that or like it was in mockup(with dark grey) to distinguish it from primary buttons
  • Since we have moved to flat design, it might be good to remove shadow for buttons as well like in the mockup
  • Ripples are not visible for other buttons other than 'Enrol in ${program}'
  • For design consistency good to keep the headers outside - hence can move the 'Summary' heading outside of the gray bg - thats how it was there before this card
  • If possible we can do this: In the mockups for all buttons they have made the bg light, and text darker. Currently for the buttons not touched like 'New Program Visit', 'Scheduled' we are having darker bg and white color text. Since the overall theme in the mockups is based on using a lighter theme and colors for the app, we can do this if possible.

@mahalakshme mahalakshme moved this from In QA to QA Failed in Avni Product May 17, 2024
@petmongrels petmongrels moved this from QA Failed to Code Review Ready in Avni Product May 17, 2024
@mahalakshme
Copy link
Contributor Author

@petmongrels I dont see any recent commit to check

@petmongrels petmongrels moved this from Code Review Ready to In Progress in Avni Product May 17, 2024
@petmongrels
Copy link
Contributor

comments

  • point 2, I have made the change but I am not able to make out whether the color is showing correctly or not visibly
  • Have changed them to use TouchableNativeFeedback. I can verify on old style buttons, but in new style I cannot make out due to the colors used
  • last one will have impact outside SubjectDashboard, so I have left it as it is

@petmongrels petmongrels moved this from In Progress to Code Review Ready in Avni Product May 17, 2024
@mahalakshme mahalakshme moved this from Code Review Ready to In Code Review in Avni Product May 17, 2024
@mahalakshme
Copy link
Contributor Author

mahalakshme commented May 17, 2024

@petmongrels

  • I see the primary and secondary buttons have the same color
  • I checked for ripple in 'Add relative' button since it is bigger(similar size as 'Enrol in') and outside of gray bg - but it was not visible.
  • this is not needed for showcase: summary card is not of card structure

Image

@mahalakshme mahalakshme moved this from In Code Review to QA Failed in Avni Product May 17, 2024
@petmongrels petmongrels removed their assignment May 17, 2024
@himeshr himeshr self-assigned this May 23, 2024
@himeshr himeshr moved this from QA Failed to Code Review Ready in Avni Product May 23, 2024
himeshr added a commit that referenced this issue May 23, 2024
@petmongrels petmongrels moved this from Code Review Ready to In Code Review in Avni Product May 27, 2024
@petmongrels petmongrels moved this from In Code Review to QA Ready in Avni Product May 27, 2024
@AchalaBelokar AchalaBelokar moved this from QA Ready to In QA in Avni Product May 29, 2024
@AchalaBelokar AchalaBelokar moved this from In QA to Done in Avni Product May 31, 2024
@AchalaBelokar
Copy link

AchalaBelokar commented May 31, 2024

  • Program enrolment audit details need to display in the Enrolment details section not in the Summary section
  • Secondary buttons like Exit, cancel we were displaying in red text before - it is good to do that or like it was in mockup(with dark grey) to distinguish it from primary buttons
  • Since we have moved to flat design, it might be good to remove shadow for buttons as well like in the mockup
  • Ripples are not visible for other buttons other than 'Enrol in ${program}'
  • There is some grey footer unnecessarily
  • When clicking on Menu -> dashboard, the filter icon should not be in the top right since the filters are not applicable for all dashboards, Hence it needs to display below the horizontal list of dashboards we display and above the selected filte
  • all senaro are tested in the card

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

5 participants