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

i have added footer #35

Closed
wants to merge 1 commit into from
Closed

Conversation

shivamy009
Copy link

@shivamy009 shivamy009 commented Oct 2, 2024

Screenshot 2024-10-02 204540
I have added this footer.
I have made another jsx file for footer.
I have added tailwind css also

problem no. -#15

Copy link

netlify bot commented Oct 2, 2024

Deploy Preview for delightful-daifuku-a9f6ea ready!

Name Link
🔨 Latest commit 003c303
🔍 Latest deploy log https://app.netlify.com/sites/delightful-daifuku-a9f6ea/deploys/66fd63df8cd2140008bd1a09
😎 Deploy Preview https://deploy-preview-35--delightful-daifuku-a9f6ea.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

🎉 Thank you @shivamy009 for your contribution! Your pull request has been submitted successfully. A maintainer will review it as soon as possible. We appreciate your support in making this project better.

@AlfiyaSiddique
Copy link
Owner

@shivamy009
Some improvements needed:

  1. First of all you have copy pasted the Foodies content of the Footer which is irrelevant.
    In the Footer write something related to TastyTrails You can use Chatgpt for this.
  2. We do not have a newsletter so remove that part.
  3. In Follow Us add Icons and only add the GitHub as we are not on facebook, Instagram, or twitter. Using text ask the visitor users to contribute.
  4. On clicking About it should scroll up to the About Us section and as we do not have a blog and contact page so remove it as well from the footer.
  5. Even the copyright has the Foodies name please fix this.

@AlfiyaSiddique
Copy link
Owner

Adding some more improvement in the code fix that as well.

@AlfiyaSiddique AlfiyaSiddique added the level 2 GSSOC level 2 Contribution - 25 points label Oct 3, 2024
@@ -3,7 +3,7 @@
}

#root {
max-width: 1366px;
/* max-width: 1366px; */
Copy link
Owner

Choose a reason for hiding this comment

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

Do not left old code on comment out remove it Completely.

@@ -45,7 +46,9 @@ function App() {
<Route path="/user/:id/update/recipe" element={<UpdateRecipe />} />
<Route path="/recipe/:id" element={<OneRecipe/>} />
</Routes>
<Footer />
{/* <Footer /> */}
<Footer1 />
Copy link
Owner

Choose a reason for hiding this comment

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

Instead of making another footer file why don't you add the new footer code to the old file only?
Please do this. And Remove Footer1.jsx
Try to add your code to existing files only unless it does not need a new file for a component or page

@shivamy009
Copy link
Author

ok i will fix all these problem

@AlfiyaSiddique
Copy link
Owner

@shivamy009 any updates ?

@harish08102004
Copy link

Hey @AlfiyaSiddique If he is not working on this issue may be I can work in this.

@AlfiyaSiddique
Copy link
Owner

@harish08102004 go ahead , assigning this issue to you.

@AlfiyaSiddique
Copy link
Owner

@harish08102004 please comment below issue #15 so that I can assign it to you.

@AlfiyaSiddique
Copy link
Owner

@harish08102004 are you willing to work on #15 ?

@harish08102004
Copy link

Hey i can work

@AlfiyaSiddique
Copy link
Owner

@harish08102004 then please make a comment on issue #15 , I need to assign this issue to you in order you to work on it.

@AlfiyaSiddique AlfiyaSiddique mentioned this pull request Oct 11, 2024
@AlfiyaSiddique AlfiyaSiddique removed gssoc-ext For GSSOC Extended Contribution. level 2 GSSOC level 2 Contribution - 25 points hacktoberfest-accepted labels Oct 11, 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.

3 participants