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

Animal page responsive #280 #362

Closed
wants to merge 7 commits into from

Conversation

Riddhi162
Copy link
Contributor

@Riddhi162 Riddhi162 commented May 19, 2024

Description

[Please include a brief description of the changes or features added]
I have made the page responsive also improved the font positioning and size etc.

Related Issue

[Cite any related issue(s) this pull request addresses. If none, simply state "None"]

Screenshots / GIFs (if applicable)

[Attach any relevant screenshots or GIFs demonstrating the changes]
Screenshot 2024-05-19 122923
Screenshot 2024-05-19 122929
Screenshot 2024-05-19 122759

Checklist:

  • I have performed a self-review of my code
  • I have added/updated relevant documentation (if needed)
  • I have tested the changes locally and they function as expected
  • I have ensured my code follows the project's coding standards

Additional Notes

[Add any additional notes or context about the changes made]

@Riddhi162 Riddhi162 changed the title Riddhipatch2 Animal page responsive May 19, 2024
@Riddhi162 Riddhi162 changed the title Animal page responsive Animal page responsive #280 May 19, 2024
@@ -101,10 +101,11 @@ function AnimalDetailsPage({
}, [formData.predictedAnimal]);
return (
// A MASTER CONTAINER
<div>
<div className="">
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

whats this empty classname for ?

@@ -443,51 +477,65 @@ function AnimalDetailsPage({
<div className="bg-white bg-opacity-57 shadow-lg ring-1 ring-gray-300 flex flex-col gap-2 justify-center border-1 p-5 rounded-3xl backdrop-blur-[6px]">
<label>
{" "}
<p className="font-extrabold text-indigo-900 pl-5 text-[1.4rem] leading-normal tracking-wider">
Describe Severity:

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change it to describe severity

@codewarnab
Copy link
Collaborator

@Riddhi162 you have to create two separate pr from two separate brances for the two pages contact information and animal details page
also dont forget to mention the issue in decsription

@Riddhi162
Copy link
Contributor Author

ohhh ok ill make the necessary changes.

@codewarnab
Copy link
Collaborator

this one is the old version of #416 right ? if yes please close the pr

@Riddhi162
Copy link
Contributor Author

ok will do so

@Riddhi162 Riddhi162 closed this May 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants