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

Docs improvements #2935

Merged
merged 6 commits into from
Sep 25, 2024
Merged

Docs improvements #2935

merged 6 commits into from
Sep 25, 2024

Conversation

ptomato
Copy link
Collaborator

@ptomato ptomato commented Sep 11, 2024

Various improvements to the docs, mostly taken from a sweep of the issue backlog. Also corrected other mistakes I ran into.

docs/duration.md Outdated Show resolved Hide resolved
…ters

Rename the parameters to "item" to match the spec text. Make clear that
they can be a string, property bag, or instance, but not other types like
number, bigint, etc.

Closes: #1394
…endar

These types don't exist anymore. Having them in the diagrams suggests that
they do.

Not sure why the "width:auto;" is necessary after editing object-model.svg
but without that, it shows up huge in the HTML page.
These were using removed APIs and so had broken. Because they're
interactive, they're not tested in CI.
This example shows how to set properties of an HTML input element using
Temporal objects. For discoverability, link to a later example that shows
how to do the reverse.

See: #1576
It wasn't clear what "these extensions" referred to from the text, and the
IETF process is finished now.
Copy link

codecov bot commented Sep 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.29%. Comparing base (82ad54f) to head (f09c3d2).
Report is 203 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2935      +/-   ##
==========================================
+ Coverage   96.51%   97.29%   +0.77%     
==========================================
  Files          23       22       -1     
  Lines       12432    10899    -1533     
  Branches     2258     1849     -409     
==========================================
- Hits        11999    10604    -1395     
+ Misses        374      295      -79     
+ Partials       59        0      -59     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

@gibson042 gibson042 left a comment

Choose a reason for hiding this comment

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

LGTM!

@ptomato ptomato merged commit a7b6c34 into main Sep 25, 2024
10 checks passed
@ptomato ptomato deleted the docs-improvements branch September 25, 2024 19:36
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.

2 participants