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

Engineers/Technical Professionals Component #26

Merged

Conversation

Raiden-Griff
Copy link
Collaborator

@Raiden-Griff Raiden-Griff commented Jan 21, 2025

Screenshot 2025-01-25 at 5 53 25 PM Fixed code from css to tailwind css Got rid of unnecessary files I had attached to branch I was unsure if the header and text should be higher or not Please let me know if I need to update anything.

@Raiden-Griff Raiden-Griff self-assigned this Jan 21, 2025
@Raiden-Griff Raiden-Griff linked an issue Jan 21, 2025 that may be closed by this pull request
Copy link
Contributor

@stormyy00 stormyy00 left a comment

Choose a reason for hiding this comment

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

i don't see the component pushed just package.json

@Raiden-Griff
Copy link
Collaborator Author

I just fixed the pull request, so I pushed the component as well. I posted it in discord as well.

height: 2px;
background-color: #ffb81c;
margin-left: 5px;
vertical-align: text-top;
Copy link
Contributor

Choose a reason for hiding this comment

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

use tailwind css. we are keeping the code consistent so css is not allowed

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I just changed it to tailwind css. I also deleted unnecessary files, as a few I attached the branch were not needed.

Copy link
Collaborator Author

@Raiden-Griff Raiden-Griff left a comment

Choose a reason for hiding this comment

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

Got rid of files that were unnecessary because they were filled with empty information. My code is now in the correct spot. I fixed it from css to tailwind css as well.

@stormyy00 stormyy00 merged commit 66e446c into dev Jan 28, 2025
5 checks passed
@stormyy00 stormyy00 deleted the armaniegonzalez/engineers-technical-professional-component branch January 28, 2025 00:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Engineers/ Technical Professionals Component
2 participants