Skip to content

Improve area editing#1012

Merged
vnugent merged 30 commits into
developfrom
fix-946
Dec 1, 2023
Merged

Improve area editing#1012
vnugent merged 30 commits into
developfrom
fix-946

Conversation

@vnugent

@vnugent vnugent commented Nov 5, 2023

Copy link
Copy Markdown
Contributor

feat: create a separate area edit page to simplify the current area page

What type of PR is this?(check all applicable)

  • refactor
  • feature
  • bug fix
  • documentation
  • optimization
  • other

Description

Related Issues

Issue #946

What this PR achieves

  • Migrate area page to Next 13 app dir
  • Move edit logic a separate page, reduce complexity of the view-only area page
  • Create a dashboard-like editing experience
  • Fix most of next/image bugs in area/climb page montage and user gallery (didn't touch recent media on main page)
  • Improve photo uploading and tagging (clean up code as well as increase responsiveness - users should see newly uploaded photos added area page)
  • Redirect /crag/<uuid> to /area/<uuid> (dead code to be removed in feature PRs)
  • Improve area page layout and UI, making Edit the most important call-to-action follows by upload photos.

Screenshots, recordings

New area page

Screenshot 2023-12-01 at 11 53 13 AM

Area edit dashboard

Screenshot 2023-12-01 at 12 01 41 PM

General continued

Screenshot 2023-12-01 at 12 05 26 PM

@vercel

vercel Bot commented Nov 5, 2023

Copy link
Copy Markdown

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

Name Status Preview Updated (UTC)
open-tacos ✅ Ready (Inspect) Visit Preview Dec 1, 2023 10:53pm

@vnugent vnugent changed the title WIP: move area page to app dir WIP: Improve area editing Nov 5, 2023
@vnugent

vnugent commented Nov 9, 2023

Copy link
Copy Markdown
Contributor Author

WIP
Screenshot 2023-11-09 at 2 43 45 PM

@vnugent vnugent changed the title WIP: Improve area editing Improve area editing Dec 1, 2023
style update to profile btn
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.

1 participant