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

Update Button.tsx #65

Closed
wants to merge 1 commit into from
Closed

Conversation

pythonistical
Copy link
Contributor

Creating the new main button style at button.tsx

Description

This PR fixes #42

Creating the new main button style
Copy link
Contributor

@shivay-at-pieces shivay-at-pieces left a comment

Choose a reason for hiding this comment

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

Hi @pythonistical this is a TypeScript file and you can't use html code directly here. You need to change the entire code to be having tsx notation.

@pythonistical
Copy link
Contributor Author

@shivay-at-pieces actually I don't understand typescript, how about renaming the file to .htm or .css ?

@shivay-at-pieces
Copy link
Contributor

Hey @pythonistical unfortunately we can't replace a file like this. It has to follow the structure being followed throughout the rest of the codebase. I will recommend if you can spend some time learning TypeScript and in the meanwhile explore some of the other issues.

This is a good starting point: https://www.typescriptlang.org/docs/handbook/intro.html

Till then we can close this PR if it is alright @jordan-pieces.

@jwafu
Copy link
Contributor

jwafu commented Jan 17, 2024

@pythonistical @shivay-at-pieces going to close this for now, but if we have more need for specific topics like this maybe we can put together some short learning that will help you (and others) in learning and contributing to this repository and other projects in the OSP 👍

lets setup a discussion around this if we want to take this further for now :) but i will be closing this PR

@jwafu jwafu closed this Jan 17, 2024
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.

Upgrade Buttons with Hover Styles
3 participants