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

Fix typos #67304

Open
wants to merge 7 commits into
base: trunk
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions backport-changelog/readme.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ The filename is the Core PR number.

For example, if your Core PR number is `1234` and is slated to be part of the WordPress 6.9 release, the filename will be `1234.md`, and will be placed in the `/backport-changelog/6.9` directory.

The content of the markdown file should be the Github URL of the Core PR, followed by a list of Gutenberg PR Github URLs whose changes are backported in the Core PR.
The content of the markdown file should be the GitHub URL of the Core PR, followed by a list of Gutenberg PR GitHub URLs whose changes are backported in the Core PR.

A single Core PR may contain changes from one or multiple Gutenberg PRs.

Expand Down Expand Up @@ -51,7 +51,7 @@ For the backport changelog, Gutenberg uses individual files as opposed to a sing

Some Gutenberg PRs may be flagged as needing a core backport PR when they don't, for example when the PR contains minor comment changes, or the changes already exist in Core.

For individual PRs, there are two Github labels that can be used to exclude a PR from the backport changelog CI check:
For individual PRs, there are two GitHub labels that can be used to exclude a PR from the backport changelog CI check:

- `Backport from WordPress Core` - Indicates that the PR is a backport from WordPress Core and doesn't need a Core PR.
- `No Core Sync Required` - Indicates that any changes do not need to be synced to WordPress Core.
Expand Down
2 changes: 1 addition & 1 deletion bin/api-docs/gen-block-lib-list.js
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ function processObjWithInnerKeys( obj ) {
*/
function augmentSupports( supports ) {
if ( 'color' in supports ) {
// If backgroud or text is not specified (true or false)
// If background or text is not specified (true or false)
// then add it as true.a
if ( ! ( 'background' in supports.color ) ) {
supports.color.background = true;
Expand Down
8 changes: 4 additions & 4 deletions bin/plugin/commands/changelog.js
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ const LABEL_TYPE_MAPPING = {
};

/**
* Mapping of label names to arbitary features in the release notes.
* Mapping of label names to arbitrary features in the release notes.
*
* Mapping a given label to a feature will guarantee it will be categorised
* under that feature name in the changelog within each section.
Expand Down Expand Up @@ -274,7 +274,7 @@ function mapLabelsToFeatures( labels ) {
*
* @param {string[]} labels Label names.
*
* @return {boolean} whether or not the issue's is labbeled as block specific
* @return {boolean} whether or not the issue's is labeled as block specific
*/
function getIsBlockSpecificIssue( labels ) {
return !! labels.find( ( label ) => label.startsWith( '[Block] ' ) );
Expand Down Expand Up @@ -343,7 +343,7 @@ function getIssueFeature( issue ) {

// 1. Prefer explicit mapping of label to feature.
if ( featureCandidates.length ) {
// Get occurances of the feature labels.
// Get occurrences of the feature labels.
const featureCounts = featureCandidates.reduce(
/**
* @param {Record<string,number>} acc Accumulator
Expand Down Expand Up @@ -941,7 +941,7 @@ function skipCreatedByBots( pullRequests ) {
}

/**
* Produces the formatted markdown for the contributor props seciton.
* Produces the formatted markdown for the contributor props section.
*
* @param {IssuesListForRepoResponseItem[]} pullRequests List of pull requests.
*
Expand Down
6 changes: 3 additions & 3 deletions bin/plugin/commands/test/__snapshots__/changelog.js.snap
Original file line number Diff line number Diff line change
Expand Up @@ -195,7 +195,7 @@ exports[`getChangelog verify that the changelog is properly formatted 1`] = `
- Tidying \`CircularOptionPicker.Option\`. ([54903](https://github.com/WordPress/gutenberg/pull/54903))

#### Typography
- Font Library: Syntax refactor repace strpos with str_contains. ([54832](https://github.com/WordPress/gutenberg/pull/54832))
- Font Library: Syntax refactor replace strpos with str_contains. ([54832](https://github.com/WordPress/gutenberg/pull/54832))
- Font Library: Use snake_case instead of camelCase on fontFamilies endpoint param. ([54977](https://github.com/WordPress/gutenberg/pull/54977))

#### Block Editor
Expand All @@ -221,7 +221,7 @@ exports[`getChangelog verify that the changelog is properly formatted 1`] = `
- Don’t use TypeScript files in scripts package. ([54856](https://github.com/WordPress/gutenberg/pull/54856))
- ESLint: Update eslint-plugin-testing-library to v6. ([54910](https://github.com/WordPress/gutenberg/pull/54910))
- Fix end-to-end test: ”WP Editor Meta Boxes > Should save the changes”. ([51884](https://github.com/WordPress/gutenberg/pull/51884))
- Font Library: Avoid deprected error in test. ([54802](https://github.com/WordPress/gutenberg/pull/54802))
- Font Library: Avoid deprecated error in test. ([54802](https://github.com/WordPress/gutenberg/pull/54802))
t-hamano marked this conversation as resolved.
Show resolved Hide resolved
- Make \`editor.getBlocks\` to return only testing-related properties. ([54901](https://github.com/WordPress/gutenberg/pull/54901))
- Migrate 'Global styles sidebar' test to Playwright. ([55045](https://github.com/WordPress/gutenberg/pull/55045))
- Migrate 'iframed block editor settings styles' tests to Playwright. ([55014](https://github.com/WordPress/gutenberg/pull/55014))
Expand Down Expand Up @@ -255,7 +255,7 @@ exports[`getChangelog verify that the changelog is properly formatted 1`] = `
- Background support: Backport fix for undefined array key. ([54850](https://github.com/WordPress/gutenberg/pull/54850))

#### Typography
- Revert "Font Library: Avoid rendering font library ui outisde gutenberg plugin". ([54947](https://github.com/WordPress/gutenberg/pull/54947))
- Revert "Font Library: Avoid rendering font library ui outside gutenberg plugin". ([54947](https://github.com/WordPress/gutenberg/pull/54947))

#### Commands
- Make the reset styles command consistent. ([54841](https://github.com/WordPress/gutenberg/pull/54841))
Expand Down
2 changes: 1 addition & 1 deletion bin/plugin/commands/test/changelog.js
Original file line number Diff line number Diff line change
Expand Up @@ -260,7 +260,7 @@ describe( 'getIssueFeature', () => {
name: '[Package] This package',
},
{
name: '[Feature] Cool Feature', // Should have priority despite prescence of block specific label.
name: '[Feature] Cool Feature', // Should have priority despite presence of block specific label.
},
{
name: '[Package] Another One',
Expand Down
4 changes: 2 additions & 2 deletions docs/contributors/code/auto-cherry-picking.md
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ Fetching commit IDs... Done!
Trying to cherry-pick one by one...

Cherry-picking round 1:
cherry-pick commit: afe9b757b4 for PR: #41198 – Site Editor: Set min-width for...
cherry-pick commit: safe9b757b4 for PR: #41198 – Site Editor: Set min-width for...
Cherry-picking finished!

Summary:
Expand All @@ -60,7 +60,7 @@ Either way, here's what happens once you proceed past the cherry-picking stage:
```
Pushing to origin/wp/6.2
Commenting and removing labels...
41198: I just cherry-picked this PR to the wp/6.2 branch to get it included in the next release: afe9b757b4
41198: I just cherry-picked this PR to the wp/6.2 branch to get it included in the next release: safe9b757b4
Done!
```

Expand Down
2 changes: 1 addition & 1 deletion docs/contributors/code/release.md
Original file line number Diff line number Diff line change
Expand Up @@ -234,7 +234,7 @@ It's important to check that:
- the plugin from the directory works as expected
- the ZIP contents (see [Downloads](https://plugins.trac.wordpress.org/browser/gutenberg/)) looks correct (doesn't have anything obvious missing)
- the [Gutenberg SVN repo](https://plugins.trac.wordpress.org/browser/gutenberg/) has two new commits (see [the log](https://plugins.trac.wordpress.org/browser/gutenberg/)):
- the `trunk` folder should have "Commiting version X.Y.Z"
- the `trunk` folder should have "Committing version X.Y.Z"
- there is a new `tags/X.Y.Z` folder with the same contents as `trunk` whose latest commit is "Tagging version X.Y.Z"

Most likely, the tag folder couldn't be created. This is a [known issue](https://plugins.trac.wordpress.org/browser/gutenberg/) that [can be fixed manually](https://github.com/WordPress/gutenberg/issues/55295#issuecomment-1759292978).
Expand Down
8 changes: 4 additions & 4 deletions docs/explanations/architecture/assets/modules.excalidraw
Original file line number Diff line number Diff line change
Expand Up @@ -158,12 +158,12 @@
"groupIds": [],
"strokeSharpness": "sharp",
"boundElementIds": [
"qYtX-Dbe48WGy5NUs2SWN",
"qYtX-Be48WGy5NUs2SWN",
"bpFYMjIuLwNHKYoF5sP1m",
"zBZdXkTFmJO9KXJqxovCk",
"zDwPJzeKWLvw4E2gXDD7h",
"tF0J9QyFKzLm643LknUSp",
"2QB5PnOHSjpZxpfsPWK9N",
"2QB5OnOHSjpZxpfsPWK9N",
"d0bomk6ntAhDkzr1jutsJ",
"vjN8YKa1-pss_8nUBz9sE",
"KF9YfK1TYP8afMie5CvX_",
Expand Down Expand Up @@ -371,7 +371,7 @@
"groupIds": [],
"strokeSharpness": "sharp",
"boundElementIds": [
"qYtX-Dbe48WGy5NUs2SWN",
"qYtX-Be48WGy5NUs2SWN",
"-rW8RlMAg9krSeGI2WgDY",
"bpFYMjIuLwNHKYoF5sP1m",
"9yv_MGYQYvx5-mDLvir57",
Expand Down Expand Up @@ -1244,7 +1244,7 @@
"groupIds": [],
"strokeSharpness": "sharp",
"boundElementIds": [
"qYtX-Dbe48WGy5NUs2SWN",
"qYtX-Be48WGy5NUs2SWN",
"-rW8RlMAg9krSeGI2WgDY",
"bpFYMjIuLwNHKYoF5sP1m",
"9yv_MGYQYvx5-mDLvir57",
Expand Down
2 changes: 1 addition & 1 deletion docs/manifest.json
Original file line number Diff line number Diff line change
Expand Up @@ -517,7 +517,7 @@
},
{
"title": "Understanding global state, local context and derived state",
"slug": "undestanding-global-state-local-context-and-derived-state",
"slug": "understanding-global-state-local-context-and-derived-state",
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm concerned that changing this slug may have ramifications for syncing with the Block Editor Handbook.

cc @ndiego @juanmaguitar

Copy link
Contributor

Choose a reason for hiding this comment

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

This may have implications for the published handbook, so let's revert it for now.

"markdown_source": "../docs/reference-guides/interactivity-api/core-concepts/undestanding-global-state-local-context-and-derived-state.md",
"parent": "core-concepts"
},
Expand Down
2 changes: 1 addition & 1 deletion docs/reference-guides/block-api/block-bindings.md
Original file line number Diff line number Diff line change
Expand Up @@ -148,7 +148,7 @@ The function to register a custom source is `registerBlockBindingsSource( args )

- `args`: `object` with the following structure:
- `name`: `string` with the unique and machine-readable name.
- `label`: `string` with the human readable name of the custom source. In case it was defined already on the server, the server label will be overriden by this one, in that case, it is not recommended to be defined here. (optional)
- `label`: `string` with the human readable name of the custom source. In case it was defined already on the server, the server label will be overridden by this one, in that case, it is not recommended to be defined here. (optional)
- `usesContext`: `array` with the block context that the custom source may need. In case it was defined already on the server, it should not be defined here. (optional)
- `getValues`: `function` that retrieves the values from the source. (optional)
- `setValues`: `function` that allows updating the values connected to the source. (optional)
Expand Down
2 changes: 1 addition & 1 deletion docs/reference-guides/data/data-core-block-editor.md
Original file line number Diff line number Diff line change
Expand Up @@ -521,7 +521,7 @@ _Properties_

- _name_ `string`: The type of block.
- _attributes_ `?Object`: Attributes to pass to the newly created block.
- _attributesToCopy_ `?Array<string>`: Attributes to be copied from adjecent blocks when inserted.
- _attributesToCopy_ `?Array<string>`: Attributes to be copied from adjacent blocks when inserted.

### getDraggedBlockClientIds

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -175,7 +175,7 @@ store( 'myFruitPlugin', {

The derived state, regardless of whether it derives from the global state, local context, or both, can also be processed on the server by the Server Directive Processing.

_Please, visit the [Understanding global state, local context and derived state](./undestanding-global-state-local-context-and-derived-state.md) guide to learn more about how derived state works in the Interactivity API._
_Please, visit the [Understanding global state, local context and derived state](./understanding-global-state-local-context-and-derived-state.md) guide to learn more about how derived state works in the Interactivity API._
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's bring this back and address it in a follow-up.


### Derived state that can be defined statically

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ These are the core principles of TypeScript's interaction with the Interactivity

- **Inferred client types**: When you create a store using the `store` function, TypeScript automatically infers the types of the store's properties (`state`, `actions`, etc.). This means that you can often get away with just writing plain JavaScript objects, and TypeScript will figure out the types for you.
- **Explicit server types**: When dealing with data defined on the server, like local context or the initial values of the global state, you can explicitly define its types to ensure that everything is correctly typed.
- **Mutiple store parts**: Even if your store is split into multiple parts, you can define or infer the types of each part of the store and then merge them into a single type that represents the entire store.
- **Multiple store parts**: Even if your store is split into multiple parts, you can define or infer the types of each part of the store and then merge them into a single type that represents the entire store.
- **Typed external stores**: You can import typed stores from external namespaces, allowing you to use other plugins' functionality with type safety.

## Installing `@wordpress/interactivity` locally
Expand Down Expand Up @@ -269,7 +269,7 @@ That's it! Now you can access the context properties with the correct types.

The derived state is data that is calculated based on the global state or local context. In the client store definition, it is defined using a getter in the `state` object.

_Please, visit the [Understanding global state, local context and derived state](./undestanding-global-state-local-context-and-derived-state.md) guide to learn more about how derived state works in the Interactivity API._
_Please, visit the [Understanding global state, local context and derived state](./understanding-global-state-local-context-and-derived-state.md) guide to learn more about how derived state works in the Interactivity API._
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's bring this back and address it in a follow-up.


Following our previous example, let's create a derived state that is the double of our counter.

Expand Down Expand Up @@ -495,7 +495,7 @@ There's something to keep in mind when when using asynchronous actions. Just lik
counter: 0,
},
actions: {
*delayedReturn(): Generator< uknown, number, uknown > {
*delayedReturn(): Generator< unknown, number, unknown > {
yield new Promise( ( r ) => setTimeout( r, 1000 ) );
return state.counter; // Now this is correctly inferred.
},
Expand Down
2 changes: 1 addition & 1 deletion docs/reference-guides/interactivity-api/iapi-about.md
Original file line number Diff line number Diff line change
Expand Up @@ -186,7 +186,7 @@ Additionally, with a standard, **WordPress can absorb the maximum amount of comp

_Complexities absorbed by the standard_

<img alt="Two columns table comparing some aspects with and without a standard. Without a standard, block developers have to take care of everything, while having a standard. Totally handled by the standard: Tooling, hydration, integrating it with WordPress, SSR of the interactive parts, inter-block communication, and frontend performance. Partially handled: Security, accessibility, and best practices. Developer responsibility: Block logic. In the without a standard column, everything is under the developer responsability." width=60% src="https://make.wordpress.org/core/files/2023/03/standard-graph.png">
<img alt="Two columns table comparing some aspects with and without a standard. Without a standard, block developers have to take care of everything, while having a standard. Totally handled by the standard: Tooling, hydration, integrating it with WordPress, SSR of the interactive parts, inter-block communication, and frontend performance. Partially handled: Security, accessibility, and best practices. Developer responsibility: Block logic. In the without a standard column, everything is under the developer responsibility." width=60% src="https://make.wordpress.org/core/files/2023/03/standard-graph.png">


With this absorption, less knowledge is required to create interactive blocks, and developers have fewer decisions to worry about.
Expand Down
2 changes: 1 addition & 1 deletion lib/class-wp-theme-json-gutenberg.php
Original file line number Diff line number Diff line change
Expand Up @@ -3415,7 +3415,7 @@ protected static function should_override_preset( $theme_json, $path, $override

/**
* Returns the default slugs for all the presets in an associative array
* whose keys are the preset paths and the leafs is the list of slugs.
* whose keys are the preset paths and the leaves is the list of slugs.
*
* For example:
*
Expand Down
8 changes: 4 additions & 4 deletions lib/experimental/media/load.php
Original file line number Diff line number Diff line change
Expand Up @@ -300,24 +300,24 @@ function gutenberg_add_crossorigin_attributes( string $html ): string {

$processor->set_bookmark( 'resume' );

$seeked = false;
$sought = false;

$crossorigin = $processor->get_attribute( 'crossorigin' );

$url = $processor->get_attribute( $tags[ $tag ] );

if ( is_string( $url ) && ! str_starts_with( $url, $site_url ) && ! str_starts_with( $url, '/' ) && ! is_string( $crossorigin ) ) {
if ( 'SOURCE' === $tag ) {
$seeked = $processor->seek( 'audio-video-parent' );
$sought = $processor->seek( 'audio-video-parent' );

if ( $seeked ) {
if ( $sought ) {
$processor->set_attribute( 'crossorigin', 'anonymous' );
}
} else {
$processor->set_attribute( 'crossorigin', 'anonymous' );
}

if ( $seeked ) {
if ( $sought ) {
$processor->seek( 'resume' );
$processor->release_bookmark( 'audio-video-parent' );
}
Expand Down
2 changes: 1 addition & 1 deletion lib/experimental/sync/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

The signaling server allows multiple clients to exchange messages with each other through various communication topics.

Topics are not defined upfront, but clients define them by subscribing to them. By subscribing to a given topic, the client tells the server to keep track of its unread messages in the given topic. By unsubscribing from a topic, the client tells the server to free the bookeeping it maintains for the given client and topic.
Topics are not defined upfront, but clients define them by subscribing to them. By subscribing to a given topic, the client tells the server to keep track of its unread messages in the given topic. By unsubscribing from a topic, the client tells the server to free the bookkeeping it maintains for the given client and topic.

Every client communicates with the server via `GET` or `POST`. Clients must have a unique identifier, which can be randomly generated. This identifier should be included as a parameter named `subscriber_id` in every request.

Expand Down
4 changes: 2 additions & 2 deletions packages/babel-plugin-makepot/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ const REGEXP_TRANSLATOR_COMMENT = /^\s*translators:\s*([\s\S]+)/im;

/**
* Given an argument node (or recursed node), attempts to return a string
* represenation of that node's value.
* representation of that node's value.
*
* @param {Object} node AST node.
*
Expand Down Expand Up @@ -265,7 +265,7 @@ module.exports = () => {
);
}

// Attempt to exract nplurals from header.
// Attempt to extract nplurals from header.
const pluralsMatch = (
baseData.headers[ 'plural-forms' ] || ''
).match( /nplurals\s*=\s*(\d+);/ );
Expand Down
2 changes: 1 addition & 1 deletion packages/base-styles/_variables.scss
Original file line number Diff line number Diff line change
Expand Up @@ -145,7 +145,7 @@ $radio-input-size: 16px;
$radio-input-size-sm: 24px; // Width & height for small viewports.

// Deprecated, please avoid using these.
$block-padding: 14px; // Used to define space between block footprint and surrouding borders.
$block-padding: 14px; // Used to define space between block footprint and surrounding borders.
$radius-block-ui: $radius-small;
$shadow-popover: $elevation-x-small;
$shadow-modal: $elevation-large;
Expand Down
2 changes: 1 addition & 1 deletion packages/block-editor/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@

### Enhancements

- Embed the `ObserveTyping` behavior within the `BlockList` component making to simplify instanciations of third-party block editors.
- Embed the `ObserveTyping` behavior within the `BlockList` component making to simplify instantiations of third-party block editors.

## 12.8.0 (2023-08-16)

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
svg {
fill: currentColor;

// Optimizate for high contrast modes.
// Optimize for high contrast modes.
// See also https://blogs.windows.com/msedgedev/2020/09/17/styling-for-windows-high-contrast-with-new-standards-for-forced-colors/.
@media (forced-colors: active) {
fill: CanvasText;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
svg {
fill: currentColor;

// Optimizate for high contrast modes.
// Optimize for high contrast modes.
// See also https://blogs.windows.com/msedgedev/2020/09/17/styling-for-windows-high-contrast-with-new-standards-for-forced-colors/.
@media (forced-colors: active) {
fill: CanvasText;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ export function useFocusHandler( clientId ) {
return;
}

// If an inner block is focussed, that block is resposible for
// If an inner block is focussed, that block is responsible for
// setting the selected block.
if ( ! isInsideRootBlock( node, event.target ) ) {
return;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ function StylePreview( { onPress, isActive, style, url } ) {

function onLayout() {
const columnsNum =
// To indicate scroll availabilty, there is a need to display additional half the column.
// To indicate scroll availability, there is a need to display additional half the column.
Math.floor( BottomSheet.getWidth() / MAX_ITEM_WIDTH ) + HALF_COLUMN;
setItemWidth( BottomSheet.getWidth() / columnsNum );
}
Expand Down
Loading
Loading