Skip to content

fix(site-2): styles #8288

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

Merged
merged 21 commits into from
Mar 24, 2023
Merged

fix(site-2): styles #8288

merged 21 commits into from
Mar 24, 2023

Conversation

PuruVJ
Copy link
Collaborator

@PuruVJ PuruVJ commented Feb 16, 2023

Fixes svelte-2's styles

@vercel
Copy link

vercel bot commented Feb 16, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
svelte-dev-2 ✅ Ready (Inspect) Visit Preview 💬 4 unresolved
✅ 1 resolved
Mar 23, 2023 at 2:52PM (UTC)

@vercel
Copy link

vercel bot commented Feb 16, 2023

Deployment failed with the following error:

Creating the Deployment Timed Out.

@PuruVJ PuruVJ changed the title fix(svelte-2): styles fix(site-2): styles Feb 17, 2023
@dummdidumm
Copy link
Member

Looks broken on certain screen sizes:
image
(notice the overlapping on this page)
image
(notice the missing padding on the left)

Also the dimensions of the left/right/middle are different to kit.svelte.dev . If possible it would be best to just copy the layout from kit.svelte.dev here (and eventually move it into site-kit).

ertucode and others added 5 commits March 19, 2023 15:20
…ng (#8402)

#8167 introduced the strict version of it - until this is configurable, we should use the relaxed version instead, since many a11y docs actually advise using ul/ol etc
- split logic up into "is this a contenteditable element" and depending on the outcome use either .wholeText or .data to check if an update is necessary
- add to puppeteer because jsdom does not support contenteditable
- one test is skipped it because it fails right now but helps test #5018

---------

Co-authored-by: suxin2017 <[email protected]>
Co-authored-by: Simon H <[email protected]>
Co-authored-by: Simon Holthausen <[email protected]>
@dummdidumm
Copy link
Member

Merging this to work on a spike to use components from site-kit 👍

@dummdidumm dummdidumm merged commit 339ea85 into sites Mar 24, 2023
@dummdidumm dummdidumm deleted the fix/docs-styles branch March 24, 2023 11:58
@gtm-nayan gtm-nayan mentioned this pull request Jun 17, 2023
5 tasks
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.

5 participants