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

Added global site config and a floating Give a Talk CTA #26

Merged
merged 1 commit into from
Sep 27, 2024

Conversation

TrystonPerry
Copy link
Contributor

No description provided.

@tomatrow
Copy link
Contributor

@TrystonPerry Looks pretty banger. Do you think we should add a way for them to dismiss the cta?

@TrystonPerry
Copy link
Contributor Author

@TrystonPerry Looks pretty banger. Do you think we should add a way for them to dismiss the cta?

Hmm. We could change it so it only shows on the home page. That would be simple.

Copy link
Contributor

@samholmes samholmes left a comment

Choose a reason for hiding this comment

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

Rebase onto latest master to resolve conflicts and remove any merge commits.

Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this a merge commit? Does it have multiple parents? Let's keep our commits flat for PRs and only merge PRs

Copy link
Contributor

@tomatrow tomatrow Sep 24, 2024

Choose a reason for hiding this comment

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

Sorry, I used the resolve conflicts button in github's ui. I didn't know it would do a merge.

Copy link
Contributor

Choose a reason for hiding this comment

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

@TrystonPerry Please remove c63f6b2 when you rebase

@samholmes samholmes force-pushed the floating-talk-request-widget branch from c63f6b2 to ad53b54 Compare September 27, 2024 20:55
Copy link
Contributor

@samholmes samholmes left a comment

Choose a reason for hiding this comment

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

Rebased and made some modifications for responsiveness

href={links.talkSubmissionUrl}
target="_blank"
rel="noopener noreferrer"
className="fixed flex items-center px-6 py-3 space-x-2 font-bold text-white transition-all duration-300 ease-in-out transform -translate-x-1/2 rounded-full shadow-lg bg-gradient-to-r from-blue-500 to-purple-600 sm:bottom-12 bottom-6 left-1/2 hover:from-blue-600 hover:to-purple-700 hover:scale-105 animate-pulse"
Copy link
Contributor

Choose a reason for hiding this comment

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

Included sm:bottom-12 bottom-6 for responsiveness

@samholmes samholmes merged commit 6191ff1 into master Sep 27, 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.

3 participants