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

Bring back video homepage layout #12468

Merged
merged 5 commits into from
Apr 6, 2023

Conversation

tabathadelane
Copy link
Contributor

@tabathadelane tabathadelane commented Apr 4, 2023

Questions:

  • Should we also remove the useMediaQuery hook that was introduced with the Persona Homepage? Or is this useful enough to keep around?
  • Do we still want to keep the forceLightMode and animated options on DocTile that we added for the Persona Homepage

@tabathadelane tabathadelane requested a review from a team as a code owner April 4, 2023 19:38
@tabathadelane tabathadelane requested review from sunnyzanchi and LizBaker and removed request for a team April 4, 2023 19:38
@github-actions github-actions bot added this to Hero to triage in Docs PRs and Issues Apr 4, 2023
@github-actions
Copy link

github-actions bot commented Apr 4, 2023

Hi @tabathadelane 👋

Thanks for your pull request! Your PR is in a queue, and a writer will take a look soon. We generally publish small edits within one business day, and larger edits within three days.

Gatsby Cloud will automatically generate a preview of your request, and will comment with a link when the preview is ready (usually 20 to 30 minutes).

@github-actions github-actions bot added the eng issues related to site functionality that requires engineering label Apr 4, 2023
@gatsby-cloud
Copy link

gatsby-cloud bot commented Apr 4, 2023

✅ docs-website-develop deploy preview ready

sunnyzanchi
sunnyzanchi previously approved these changes Apr 6, 2023
Copy link
Contributor

@sunnyzanchi sunnyzanchi left a comment

Choose a reason for hiding this comment

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

looks good! i think it's okay to keep the light DocTile variant/animated prop and the useMediaQuery hook around, i can imagine them being useful outside of this context

@@ -19,6 +19,7 @@ export const TOGGLE_VIEWS = {
const mobileBreakpoint = '450px';

const ToggleSelector = ({ className, showTooltip }) => {
showTooltip = true;
Copy link
Contributor

Choose a reason for hiding this comment

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

should this be here? this means it pops up even if you're already on the alternate view

Copy link
Contributor Author

Choose a reason for hiding this comment

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

nope 🤡 It wasn't even doing what i wanted it to, and i forgot i put it there

@tabathadelane tabathadelane merged commit 445e7d5 into develop Apr 6, 2023
7 checks passed
@tabathadelane tabathadelane deleted the tabatha/revert-to-video-homepage branch April 6, 2023 21:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
eng issues related to site functionality that requires engineering
Projects
Docs PRs and Issues
Hero to triage
Development

Successfully merging this pull request may close these issues.

None yet

3 participants