Skip to content

Text Shadow on Header & Get Involved Hover - #78

Open
ney927 wants to merge 4 commits into
mainfrom
header-text-shadows
Open

Text Shadow on Header & Get Involved Hover#78
ney927 wants to merge 4 commits into
mainfrom
header-text-shadows

Conversation

@ney927

@ney927 ney927 commented Aug 14, 2024

Copy link
Copy Markdown
Collaborator

No description provided.

@netlify

netlify Bot commented Aug 14, 2024

Copy link
Copy Markdown

Deploy Preview for storied-fox-9ba586 ready!

Name Link
🔨 Latest commit 30d48a3
🔍 Latest deploy log https://app.netlify.com/sites/storied-fox-9ba586/deploys/66bcd2a409835d00088bc4bf
😎 Deploy Preview https://deploy-preview-78--storied-fox-9ba586.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@HasithDeAlwis HasithDeAlwis changed the title text shadow on header Text Shadow on Header & Get Involved Hover Aug 14, 2024
const link = data?.link;

const overlayStyle = {
backgroundSize: '100% 600%',

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

can we experiment with a smaller background size for the hover? I think it might be cool if the gradient didn't span the whole component 👍

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

done. let me know if you want any edits. making the gradient not span the whole thing felt a little abrupt. so i just lessened it
image

className="relative flex flex-col w-10/12 transition-all duration-30 text-white
rounded-tr-[3rem] rounded-bl-[3rem]
hover:rounded-tr-none hover:rounded-bl-none hover:rounded-tl-[3rem] hover:rounded-br-[3rem]
bg-primaryGray h-[16rem] sm:w-[32rem] sm:h-[22rem] shadow-red-bottom-left hover:shadow-red-bottom-left-hover"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Did the shadows look ugly with the gradient?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

no its just that the reference pic from Avantika didn't have them

alt="Header Background"
/>
<div className="absolute z-10 top-[14vh] sm:top-[20vh] md:top-[18vh] w-full text-center">
<div className="absolute z-10 top-[14vh] sm:top-[20vh] md:top-[18vh] w-full text-center" style={{'text-shadow': '2px 4px 8px rgba(0, 0, 0, 0.5)'}}>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think this text shadow works well 🔥

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

tyty 🔥

<div
onMouseEnter={() => setIsHovering(true)}
onMouseLeave={() => setIsHovering(false)}
className="relative flex flex-col w-10/12 transition-all duration-30 text-white

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

little typo here (duration-30), I know that wasn't you, but could we change that to 300

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

done ✅

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.

2 participants