SIS-Team-24 / 2023-SIS-Team-24

Using Natural Language Processing, Text Insights examines and provides insights to text.
1 stars 0 forks source link

Home (Bug) Fixes #203

Closed srujankarthik closed 1 year ago

srujankarthik commented 1 year ago
srujankarthik commented 1 year ago

@JamesBug404 just added in a few more reviewers, sorry I wasn't able to type on discord cause it's buggy

JamesBug404 commented 1 year ago

image

Fixed Change font display bug, emotion "Fear" colour.

henrygoodman commented 1 year ago

Small one I had in mind, the placeholder text for the Input Textarea mentions Enter 100 words or more to summarise (or something along those lines).

Could this also be changed to Enter 100 words or more to analyse? TY

JamesBug404 commented 1 year ago

Enter 100 words or more to summarise

Done.

henrygoodman commented 1 year ago

Enter 100 words or more to summarise

Done.

That latest change only updated the div title, not the placeholder text If you ctrl-f for placeholder it should show up

Also this change seemed to have caused some regression issues in the 1st Cypress test, which seems to check font options. One of you may have to update the strings in this test case

JamesBug404 commented 1 year ago

Enter 100 words or more to summarise

Done.

That latest change only updated the div title, not the placeholder text If you ctrl-f for placeholder it should show up

Also this change seemed to have caused some regression issues in the 1st Cypress test, which seems to check font options. One of you may have to update the strings in this test case

oops, sorry my bad... :/

srujankarthik commented 1 year ago

home bug fixes