Skip to content

Feature/ratings#16

Open
madisonseal wants to merge 2 commits intodevfrom
feature/ratings
Open

Feature/ratings#16
madisonseal wants to merge 2 commits intodevfrom
feature/ratings

Conversation

@madisonseal
Copy link
Copy Markdown
Contributor

created the ratings component. has user, desc, rating (with half ratings!), and card size!

@@ -2,19 +2,29 @@ type PrimaryButtonProps = {
buttonColor: string;
borderColor: string;
textColor: string;
buttonWidth?: string;
buttonHeight?: string;
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

added props

color: textColor}}> Join Waitlist
className=" whitespace-nowrap border flex items-center justify-center rounded-4xl tracking-normal"
style={{
backgroundColor: buttonColor,
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

pretty straightforward, added props again

<span className="text-[#FA9E4D]"> Two </span><span className="text-[#52AEEF]"> Cents </span>
<header className="relative flex justify-between items-center gap-2 w-full bg-white px-7 py-6 z-40 md:z-60">
<div>
<img
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

changed text + image -> svg logo

@@ -0,0 +1,43 @@
import { Rating } from "react-simple-star-rating";
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

new component!


interface WaitListJoinProps {
imageSrc?: string;
}
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

added image src for an optional centy image ;)

<p className="text-center! font-semibold! text-2xl! md:text-4xl! tracking-normal! leading-snug! pb-6! m-0!">
Start spending and <br />
saving with{" "}
<span className="text-[#52AEEF] font-normal" style={{ fontFamily: '"gooddog-new", sans-serif' }}>
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

added props here

@@ -0,0 +1,84 @@
import WaitListJoin from "../components/WaitlistJoin";
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

new comp!

Comment thread two-cents/src/index.css
@@ -1,36 +1,20 @@
@import "tailwindcss";
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

i had a lot of issues with the index.css globally overriding my pages and my tailwind classes, i think you can restore this to the previous file as i added !important to the styling that overlaps with index.css

Comment thread two-cents/package.json
"react": "^19.2.4",
"react-dom": "^19.2.4",
"react-router-dom": "^7.13.1"
"react-router-dom": "^7.13.1",
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

keep my changes, here used this library for ratings

@@ -20,9 +20,7 @@
"strict": true,
"noUnusedLocals": true,
"noUnusedParameters": true,
"erasableSyntaxOnly": true,
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

i don't remember doing this, i keep you can restore

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.

1 participant