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

Improve Sample Test page and Voice Recorder #2442

Merged
merged 5 commits into from
Jun 3, 2022

Conversation

Ashesh3
Copy link
Member

@Ashesh3 Ashesh3 commented May 19, 2022

Closes #2418

/sample

  • Capitalized "title: "
  • Changed hover border to black, since green hover on green cards blend and do not give a immersive hover effect.

Previous Hover color:
image

Updated Hover color:
image

/sample/<sample_id>

Updated:

image

Recorder

When the user denies the browser's microphone permission, appropriate message is shown to the user and the recording indicator is stopped. It only works when the user has granted it the required permission.

  • Refer to the GIF below

ILvP8RHQb5

Side Note:
image

Seems like the above field is no longer used for a patient, In that case no new patients will have this field displayed on their profile under Sample Test page. Would it be okay to just leave it as it is then?

@Ashesh3 Ashesh3 requested a review from a team May 19, 2022 12:11
@netlify
Copy link

netlify bot commented May 19, 2022

👷 Deploy request for dreamy-fermat-b5024e pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit ba3fb4c

@nihal467
Copy link
Member

@Ashesh3 @gigincg can you check the merge conflict

@nihal467
Copy link
Member

test approved

@khavinshankar
Copy link
Member

@Ashesh3 please do work on the merge conflict

@Ashesh3 Ashesh3 requested a review from a team as a code owner June 3, 2022 05:07
@Ashesh3
Copy link
Member Author

Ashesh3 commented Jun 3, 2022

@khavinshankar Done! thanks for the ping.

@khavinshankar khavinshankar merged commit 34d8a5e into coronasafe:develop Jun 3, 2022
@Ashesh3 Ashesh3 deleted the fix-2418 branch June 3, 2022 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve Sample Test History page
3 participants