search

adarshjp / Semantic-Cueing

Semantic Cueing is a system which helps to provide a speech therapy to patients with cognitive disabilities.
http://semantic-cueing.herokuapp.com/
1 stars 1 forks source link

Create test UI Updated #116

Closed ajay-acer closed 2 years ago

ajay-acer commented 2 years ago

UI of create test is changed. 2 Tabs added and on scroll new questions are added. Do not merge this pull request

ajay-acer commented 2 years ago

screencapture-localhost-3000-create-test-2022-04-11-18_59_48

This is the output as of now.

adarshjp commented 2 years ago

Pls answer the following questions

  1. Why there is no background colour near the profile pic on the sidebar?
  2. Is this compatible with all devices?
  3. Where is the submit button?
  4. Is it ready to merge?

Also, share the ss of selected questions 🙏🏻

ajay-acer commented 2 years ago
  1. The GoFullPage is used to get screenshot, there will be no background colour there.
  2. Yes it is compatible with all devices.
  3. Submit button will come in the selected questions tab.
  4. plz pull and execute image screencapture-localhost-3000-create-test-2022-04-11-20_18_58 screencapture-localhost-3000-create-test-2022-04-11-18_59_48
ajay-acer commented 2 years ago

There are many small bugs that need to be fixed. These bugs show only sometimes not always. So let it be like this itself. Do not merge

ajay-acer commented 2 years ago

Now tabs are added to view selected and select questions. Cart Button is also added to view the number of questions selected.

screencapture-localhost-3000-create-test-2022-04-19-22_10_41

screencapture-localhost-3000-create-test-2022-04-19-22_10_03

Plz pull this branch and merge, if there are no bugs found.

adarshjp commented 2 years ago

Fix the following Issues

ajay-acer commented 2 years ago

Fix the following Issues

  • [x] Unable to see more than 5 questions
  • [ ] First scroll gives the error message (sometimes)
  • [x] Pls compare the response time by fetching 10 questions (Please use the POSTMAN collection which I have shared with you)
  • [x] Pls add a multi-select dropdown for assigning a test for multiple patients (BE Change Required )
  • [x] Add an option to clear all the selected questions

There is the option that already exits the 'Reset' button is present to clear the selected questions. Multi-select dropdown and BE changes implemented.

adarshjp commented 2 years ago

Pls go ahead with fetching 10 questions at a time

ajay-acer commented 2 years ago

Now 10 questions are fetched at a time. Plz pull this branch and merge it with the main.