Skip to content

fix: add working back button on ProjectEditor page #1187

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

fix: add working back button on ProjectEditor page #1187

wants to merge 2 commits into from

Conversation

balraj009
Copy link

Closes: #1173

Describe the add-ons or changes you've made 📃

  • Fixed the issue related to missing navigation on the New Project (ProjectEditor) page.
  • Added a Back Button in the page header to allow users to return to the previous page or dashboard without needing to manually navigate.
  • Updated ProjectEditor.jsx to include useNavigate from react-router-dom for back navigation handling.
  • Verified the button works across all screen sizes and in both light/dark modes.

Screenshots 📷

image

Type of change ☑️

  • Bug fix
  • New feature

Note to reviewers 📄

  • The back button navigates users to the previous route in history.
  • Tested on local environment with Vite + React + Tailwind, working as expected.

@vercel Vercel
Copy link

vercel bot commented Aug 11, 2025

Someone is attempting to deploy a commit to the avdheshvarshney's projects Team on Vercel.

A member of the Team first needs to authorize it.

@netlify Netlify
Copy link

netlify bot commented Aug 11, 2025 •
edited
Loading

✅ Deploy Preview for code-a2z ready!

Name Link
🔨 Latest commit 13a1411
🔍 Latest deploy log https://app.netlify.com/projects/code-a2z/deploys/689b317d0a85e1000884f3af
😎 Deploy Preview https://deploy-preview-1187--code-a2z.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 project configuration.

@github-actions GitHub Actions
Copy link

Thank you for submitting your pull request! We'll review it as soon as possible. For further communication, join our discord server https://discord.gg/tSqtvHUJzE.

@balraj009 balraj009 mentioned this pull request Aug 11, 2025
Open
4 tasks
@vercel Vercel
Copy link

vercel bot commented Aug 11, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
code-a2z ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 11, 2025 6:14pm

Avdhesh-Varshney
Avdhesh-Varshney requested changes Aug 11, 2025
View reviewed changes
Copy link
Member

@Avdhesh-Varshney Avdhesh-Varshney left a comment

Choose a reason for hiding this comment

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

image

Review it for light mode

backend/.env.example
Copy link
Member

Choose a reason for hiding this comment

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

don't delete this file

backend/server.js
Copy link
Member

Choose a reason for hiding this comment

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

Revert all changes

frontend/.env.example
Copy link
Member

Choose a reason for hiding this comment

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

Undo this file

frontend/src/components/ProjectEditor.jsx
Copy link
Member

Choose a reason for hiding this comment

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

This file is hard to review, plz keep the the indentation as before and only do your changes.

frontend/vite.config.js Outdated
Copy link
Member

Choose a reason for hiding this comment

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

Undo

@Avdhesh-Varshney
Copy link
Member

@balraj009 follow the component hover colors as of other buttons

@balraj009
Copy link
Author

@Avdhesh-Varshney ,
Now updated as per your guidance, please review it if further any changes than let me know.

@Avdhesh-Varshney
Copy link
Member

@balraj009 I think still all the changes are left as it is, plz review all the comments on each file

@Avdhesh-Varshney
Copy link
Member

@balraj009 What's the update?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Avdhesh-Varshney Avdhesh-Varshney Avdhesh-Varshney requested changes

Requested changes must be addressed to merge this pull request.

Assignees

@balraj009 balraj009

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add Back Navigation to New Project Page
2 participants
@balraj009 @Avdhesh-Varshney