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

Hide books section on home page since books are empty #781

Merged
merged 2 commits into from
Oct 24, 2023

Conversation

vyktoremario
Copy link
Contributor

@vyktoremario vyktoremario commented Oct 18, 2023

Type of change

  • Content addition
  • Bug fix
  • Behavior change

Summary of change

Hide books section on home page since books are empty

  • Books has a big featured section on the homepage but SolidJS doesn't have any. We should probably consider a change to not make books so heavily featured and switch some homepage items around. Not a blocker but noting here.
Screenshot 2023-10-18 at 10 47 53

Checklist

  • This change resolves [solidjs-tailwind] Hide books section on home page since books are empty #693
  • I have confirmed with a maintainer that this change is desired and been
    given the go-ahead to work on it in the relevant issue discussion
  • The changes follow the contributing guidelines
  • I have verified the change and the rest of the site works as expected
  • If the change introduces new components, these have been added to
    packages/system/src/components with a .stories.tsx file that
    adequately renders possible variations of each component.

@netlify
Copy link

netlify bot commented Oct 18, 2023

Deploy Preview for node-framework-dev ready!

Name Link
🔨 Latest commit 6dd5a40
🔍 Latest deploy log https://app.netlify.com/sites/node-framework-dev/deploys/6537e02748892e0008c1a280
😎 Deploy Preview https://deploy-preview-781--node-framework-dev.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.

@netlify
Copy link

netlify bot commented Oct 18, 2023

Deploy Preview for framework-dev ready!

Name Link
🔨 Latest commit 6dd5a40
🔍 Latest deploy log https://app.netlify.com/sites/framework-dev/deploys/6537e027f8e4d30008f88ad1
😎 Deploy Preview https://deploy-preview-781--framework-dev.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.

@netlify
Copy link

netlify bot commented Oct 18, 2023

Deploy Preview for qwik-framework-dev ready!

Name Link
🔨 Latest commit 6dd5a40
🔍 Latest deploy log https://app.netlify.com/sites/qwik-framework-dev/deploys/6537e0278a87050009e79818
😎 Deploy Preview https://deploy-preview-781--qwik-framework-dev.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.

@netlify
Copy link

netlify bot commented Oct 18, 2023

Deploy Preview for system-framework-dev ready!

Name Link
🔨 Latest commit 6dd5a40
🔍 Latest deploy log https://app.netlify.com/sites/system-framework-dev/deploys/6537e027ea52e000085ba258
😎 Deploy Preview https://deploy-preview-781--system-framework-dev.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.

@netlify
Copy link

netlify bot commented Oct 18, 2023

Deploy Preview for react-framework-dev ready!

Name Link
🔨 Latest commit 6dd5a40
🔍 Latest deploy log https://app.netlify.com/sites/react-framework-dev/deploys/6537e0278e37610008ba5e57
😎 Deploy Preview https://deploy-preview-781--react-framework-dev.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.

@netlify
Copy link

netlify bot commented Oct 18, 2023

Deploy Preview for vue-framework-dev ready!

Name Link
🔨 Latest commit 6dd5a40
🔍 Latest deploy log https://app.netlify.com/sites/vue-framework-dev/deploys/6537e027ac83a700083c8cc8
😎 Deploy Preview https://deploy-preview-781--vue-framework-dev.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.

@netlify
Copy link

netlify bot commented Oct 18, 2023

Deploy Preview for svelte-framework-dev ready!

Name Link
🔨 Latest commit 6dd5a40
🔍 Latest deploy log https://app.netlify.com/sites/svelte-framework-dev/deploys/6537e0271d70610009f7668b
😎 Deploy Preview https://deploy-preview-781--svelte-framework-dev.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.

@netlify
Copy link

netlify bot commented Oct 18, 2023

Deploy Preview for graphql-framework-dev ready!

Name Link
🔨 Latest commit 6dd5a40
🔍 Latest deploy log https://app.netlify.com/sites/graphql-framework-dev/deploys/6537e0272ca827000884b858
😎 Deploy Preview https://deploy-preview-781--graphql-framework-dev.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.

@netlify
Copy link

netlify bot commented Oct 18, 2023

Deploy Preview for deno-framework-dev ready!

Name Link
🔨 Latest commit 6dd5a40
🔍 Latest deploy log https://app.netlify.com/sites/deno-framework-dev/deploys/6537e02722f45f000810a620
😎 Deploy Preview https://deploy-preview-781--deno-framework-dev.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.

@netlify
Copy link

netlify bot commented Oct 18, 2023

Deploy Preview for solid-framework-dev ready!

Name Link
🔨 Latest commit 6dd5a40
🔍 Latest deploy log https://app.netlify.com/sites/solid-framework-dev/deploys/6537e027534dfd0008223c17
😎 Deploy Preview https://deploy-preview-781--solid-framework-dev.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.

@netlify
Copy link

netlify bot commented Oct 18, 2023

Deploy Preview for angular-framework-dev ready!

Name Link
🔨 Latest commit 6dd5a40
🔍 Latest deploy log https://app.netlify.com/sites/angular-framework-dev/deploys/6537e0272f1a7500083976b9
😎 Deploy Preview https://deploy-preview-781--angular-framework-dev.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.

Copy link
Contributor

@tvanantwerp tvanantwerp left a comment

Choose a reason for hiding this comment

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

Would like to know more about why some of the content was removed. It's not part of the linked ticket and I don't see an obvious reason for it.

Also, a suggestion about structuring the conditional inclusion of books.

Comment on lines -88 to -97
{
title: 'Rodney Lab',
author: 'Rodney',
description:
'Hello and welcome! I’m Rodney, a web developer based in the UK 🇬🇧 and am available for consultancy work.',
image:
'https://rodneylab.com/assets/rodney-johnson-about-rodneylab-1350x1350.208d1490.avif',
href: 'https://rodneylab.com/tags/sveltekit/',
tags: [],
},
Copy link
Contributor

Choose a reason for hiding this comment

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

Curious, why was this one removed? At first glance, it does look like a bunch of SvelteKit blogs.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was a duplicate.

So I had to remove it.

"Matija from 🇭🇷 Croatia and he's infinitely curious at how things work but he's mostly passionate about JavaScript and 🎨 UI/UX design.",
"Matija from Croatia and he's infinitely curious at how things work but he's mostly passionate about JavaScript and UI/UX design.",
Copy link
Contributor

Choose a reason for hiding this comment

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

Wondering why emoji are being removed. Rendering issues? I know flags sometimes don't render right depending on the OS and browser—had that problem plenty of times on Windows machines. Not sure if that's still the case, or the reason for removing them in this PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I removed them due to rendering issues.
They were not rendering correctly on my Mac and I perceive some users might have the same issue.

I don't think it adds enough value to decide to accommodate it, so I removed it.

Comment on lines -204 to -211
{
name: 'Svelte Summit',
description: 'A meetup dedicated to the latest release of Svelte',
image: 'https://sveltesociety.dev/images/logo.svg',
type: 'Live Events',
href: 'https://beta.guild.host/svelte-society-london/events',
tags: ['conferences'],
},
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks outdated as a conference, but appears to be a still-active community. Why remove it? Just needs some updating to make it a bit more evergreen.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was a duplicate and the link is not the correct link for Svelte Summit.
There is another entry that has the correct info.

packages/system/src/components/homepage/homepage.tsx Outdated Show resolved Hide resolved
@vyktoremario vyktoremario merged commit 052649a into main Oct 24, 2023
46 checks passed
@vyktoremario vyktoremario deleted the hide-book-section-on-homepage branch October 24, 2023 16:32
@thisdot-bot
Copy link

Hi @vyktoremario. 👋

Your PR has been approved and merged. 🎉

Thank you for your continued contributions to framework.dev. ❤️

Keep up the great work!

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.

[solidjs-tailwind] Hide books section on home page since books are empty
3 participants