Skip to content

Conversation

@tomoam
Copy link
Member

@tomoam tomoam commented Feb 21, 2024

No description provided.

Michael Baker and others added 30 commits December 13, 2023 10:21
* Update README.md

* Update +error.svelte

* Update +error.svelte

* Update README.md
Co-authored-by: Rich Harris <rich.harris@vercel.com>
Co-authored-by: Rich Harris <rich.harris@vercel.com>
It was not clear which page below the layout inherits the data, because in fact it could be any page.

The confusion can arise from the way the previous sentence was expressed, as **visually**, there is nothing `below` the layout in the project tree.

There are siblings though, like `+page.svelte` but for a Svelte beginner it is **not** immediately obvious that this page also falls `under` the same layout.
…el (sveltejs#443)

* docs: apply fail fast for keydown function handler

* style: apply fail fast for if condition
* docs: minor addition to onMount usage

* Update content/tutorial/01-svelte/07-lifecycle/01-onmount/README.md

* Update content/tutorial/01-svelte/07-lifecycle/01-onmount/README.md

---------

Co-authored-by: Rich Harris <hello@rich-harris.dev>
…art 3 / Forms / Progressive enhancement (sveltejs#461)

* include entire css styles from unsolved code to solved code

* slight nitpick; use em dash instead of hyphen for consistency
The readme is missing the line:
<div class="centered">
as well as the indentation that are present in the code editor solution.

this can cause a slight confusion while following the tutorial
in the tutorial excercise 'named slots', the 'paper.svg' has fill="none". The exercise is to build something that looks like a 'business card' on a table it seems, however you cannot see the card, so it looks more like words etched into wood!
screenshot : https://prnt.sc/kpg6vljbZT-T
Fixes bad contrast in dark mode at, e.g., /numeric-inputs
* fix: smaller sidebar footer horizontal padding

* fix: fine tune it
* Update README.md

docs: 04-updating-arrays-and-objects, add `let obj = { foo: { bar: 'old' } };` for more clarity

* Apply suggestions from code review

---------

Co-authored-by: Rich Harris <hello@rich-harris.dev>
* add aria-role:none to get rid of warning

* Apply suggestions from code review

---------

Co-authored-by: Rich Harris <hello@rich-harris.dev>
* fix: sidebar scrolls to top after navigate

* fix: sidebar scrolls to top after navigate
Co-authored-by: Rich Harris <rich.harris@vercel.com>
Co-authored-by: Rich Harris <rich.harris@vercel.com>
Co-authored-by: Rich Harris <rich.harris@vercel.com>
* changed id from const to let

* more const to let changes
…ideo (sveltejs#503)

* Change "video" to "medium", as the word should cover both audio and video

* Apply suggestions from code review

* Apply suggestions from code review

---------

Co-authored-by: Rich Harris <hello@rich-harris.dev>
* Clarify the items in the each needing ids/keys

Try to clarify that it is updating the DOM/components after deleting the last item.

For some reason, I read this as changing the array itself by removing the last item.

I suspect my changes need to be slimmed down and edited.

Yes, I am neuro-divergent. Why do you ask? 😃

* Apply suggestions from code review

---------

Co-authored-by: Rich Harris <hello@rich-harris.dev>
@vercel
Copy link

vercel bot commented Feb 21, 2024

@tomoam is attempting to deploy a commit to a Personal Account owned by @svelte-jp on Vercel.

@svelte-jp first needs to authorize it.

@vercel
Copy link

vercel bot commented Feb 21, 2024

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

Name Status Preview Comments Updated (UTC)
learn-svelte-jp ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 21, 2024 4:16am

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.