-
Notifications
You must be signed in to change notification settings - Fork 3
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
Added requested changes in add-results html #293
base: develop
Are you sure you want to change the base?
Conversation
added requested changes in add-results html
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line 19: To match design the results need to start with uppercase
Line 22: excel needs to start with uppercase to match design
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job!
Added styling to add-results and updated tailwindcss to 4.0.1
What does this PR do?
Updated requested changes.
Description of Task to be completed?
I have updated the changes in the HTML as Tomatsuppe commented below. I think I made a mistake with this when I committed, as I had to save it again to proceed. Because of that, the changes are not showing up for you. I apologize for this...
How should this be manually tested?
Go over en check with the latest comments.
Any background context you want to provide?
No.