Skip to content

Fix: resolved issue 501 improve UI/UX of the website.#573

Open
OfficialArpitNege wants to merge 2 commits into
firstcontributions:mainfrom
OfficialArpitNege:fix-issue-501
Open

Fix: resolved issue 501 improve UI/UX of the website.#573
OfficialArpitNege wants to merge 2 commits into
firstcontributions:mainfrom
OfficialArpitNege:fix-issue-501

Conversation

@OfficialArpitNege
Copy link
Copy Markdown

Fixes issue #501
Improved the UI/UX of the website.
image

@Sean-Kenneth-Doherty
Copy link
Copy Markdown

I tested this locally. It builds, but I do not think this is ready to merge as a fix for #501.

Checks run:

  • git diff --check main...HEAD passes
  • pnpm build passes

Concerns:

  • The PR includes generated .astro/* files. Those should be removed from the branch; they are local generated/cache files.
  • Most of the user-facing diff is arbitrary palette changes: the app background changes from near-black to #c28e34, cards change to a strong red background, the hover overlay becomes bright green, and tags become gray/black. That is a large visual direction change, but the PR does not explain the design goal or how it resolves the broad UI/UX issue.
  • The package.json name change may be reasonable as a separate cleanup, but it is unrelated to UI/UX and changes the package name shown in build output.
  • The navbar CSS addition is mis-indented and adds a dark background on each nav link rather than addressing a specific UX problem.

For #501, I would keep the patch focused on one concrete UX improvement at a time, with before/after screenshots or a clear description of the interaction/readability problem being solved.

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