这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@MingoHaha
Copy link
Collaborator

No description provided.

@rosuqt rosuqt self-requested a review March 18, 2025 15:28
Copy link
Owner

@rosuqt rosuqt left a comment

Choose a reason for hiding this comment

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

**Overall Verdict: Safe to merge, but have to install react-icons to resolve conflicts. Needs polishing to match other pages (will be polished by me (ally) since my fault for last minute changes sa UI. Please wait for merging thx

Conflict Changes

  • package.json - installed react-icons (might conflict with your local repos if not installed, but safe to merge. For other members please install react-icons to resolve conflicts npm install react-icons.

Polishing (ally notes)

  • remove nav bar
  • raise job search bar to the center-top of the screen
  • possible change sa sign-in/employer's sign up page cuz different designs for each pages
  • adjust other contents based on changes
  • change the left side job preview to match the figma design
  • add blur and floating sign-in redirectors
    Note to mark: feel free to do the polishing if u want let me know lang

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