Skip to content
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.

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

Why choose us section has been created #16

Conversation

drakkomaximo
Copy link

Hi @shefihu closes #9

Summary of Changes

Component Structure

  • Converted the WhyChooseUs component to a React functional component, ensuring modern React practices are followed.

Responsive Design

  • Utilized Tailwind CSS to ensure the component is responsive across different screen sizes.
  • Implemented a grid layout with grid-cols-1 md:grid-cols-3 to switch from a single-column layout on smaller screens to a three-column layout on medium and larger screens.

Hover and Interaction States

  • Added smooth hover effects using Tailwind CSS classes like hover:scale-105 and transition duration-300 to enhance user interaction.

Typography and Color Scheme

  • Ensured consistent typography using Tailwind CSS classes such as text-sm, uppercase, tracking-wider, and font-bold.
  • Applied a consistent color scheme with classes like text-gray-700 to maintain visual uniformity.

Code Simplification

  • Removed unnecessary files (CryptoPayments.tsx, InvoiceManagement.tsx, PayrollProcessing.tsx) to streamline the component structure.
  • Centralized feature data in a constant array, allowing for easy updates and maintenance.

These changes collectively enhance the component's responsiveness, interactivity, and visual consistency, aligning with modern web development standards.

2025-02-21.15-12-10.mp4

@shefihu
Copy link
Collaborator

shefihu commented Feb 22, 2025

@drakkomaximo please you have to use the style guide to handle things, you can checkout develop and pull to see the updated index.css and tailwind config, please update acccordingly

@drakkomaximo
Copy link
Author

Hello @shefihu, why did you remove my assignment from this issue? I can update the branch with the changes you requested. I don’t understand why I’m being penalized so immediately.

@shefihu
Copy link
Collaborator

shefihu commented Feb 22, 2025

@drakkomaximo i will assign you back the right way, made an error at my end

@drakkomaximo
Copy link
Author

No worries, the mistake was mine since I didn't see the develop branch, so I pushed to main. It was a misunderstanding, but I'll fix the issue. Thanks!

@drakkomaximo drakkomaximo changed the base branch from main to develop February 22, 2025 06:30
@drakkomaximo drakkomaximo changed the base branch from develop to staging February 22, 2025 06:31
@drakkomaximo drakkomaximo deleted the feature/why-choose-us-section branch February 23, 2025 23:27
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.

DefiFundr - 007: Implement "Why Choose Us" Section for Payroll Solutions
2 participants