Skip to content

[WIP][TZD-260020] Add Brand Page and Update README#21

Open
iamabdullah73524 wants to merge 3 commits into
mainfrom
feature/TZD-260020
Open

[WIP][TZD-260020] Add Brand Page and Update README#21
iamabdullah73524 wants to merge 3 commits into
mainfrom
feature/TZD-260020

Conversation

@iamabdullah73524

Copy link
Copy Markdown
Collaborator

Summary

  • Added Brand Page
  • Updated README documentation
  • Added Light Mode and Dark Mode screenshots
  • Refactored Career page openings using map()

Status

🚧 Work In Progress

Additional review and refinements may be required before merging.

Testing

  • Tested Brand page functionality
  • Tested search and filter functionality
  • Verified README updates
  • Verified Career page rendering

@iamabdullah73524

Copy link
Copy Markdown
Collaborator Author
Screenshot 2026-06-13 222200 Screenshot 2026-06-13 222207 Screenshot 2026-06-13 222303 Screenshot 2026-06-13 222311

@recursivezero recursivezero left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

  • Create reusable components to call within .map() on all places
  • Rename screenshot filename to relevant filename

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

rename file to what page name is not screenshot

Comment thread src/pages/career.astro
openings
.filter((job) => job.isActive)
.map((job) => (
<div class="dark:border-foreground/10 bg-background/60 rounded-3xl border-2 border-gray-300 p-8 shadow-sm backdrop-blur-md">

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

create this as Separate component BrandTile.astro and pass the props

do the same for other place also

@recursivezero recursivezero left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

  • Add media query for the font size of website name on header

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