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

7 #183

Open
wants to merge 67 commits into
base: main
Choose a base branch
from
Open

7 #183

wants to merge 67 commits into from

Conversation

n-ce
Copy link
Owner

@n-ce n-ce commented May 1, 2024

The point of this pr is enable link previews for https://ytify.netlify.app?s=id on the social medias.

The next stage is converting LIT components to Solid-JS. There are advantages to this such as component transparency and reduced complexity in wrapping the components around points of usage. We use Solid-JS simply as a library and not a framework in this case. Our priority is still HTML which is why the toggle-switch will continue to be a web component so that it can be written directly in HTML without any JS involved in injecting it.

The third stage would be involving breaking down the fetchList method and creating collection item components. The advantages to this is only loading collections on demand and onto list instead of their own collapsable children.

Copy link

netlify bot commented May 1, 2024

Deploy Preview for ytify ready!

Name Link
🔨 Latest commit ae1ebf1
🔍 Latest deploy log https://app.netlify.com/sites/ytify/deploys/66515f437451dd0008a825d4
😎 Deploy Preview https://deploy-preview-183--ytify.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 56
Accessibility: 79
Best Practices: 83
SEO: 76
PWA: 100
View the detailed breakdown and full score reports

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

@n-ce n-ce linked an issue May 15, 2024 that may be closed by this pull request
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.

Support Live Streams
3 participants