Skip to content
This repository was archived by the owner on Mar 14, 2024. It is now read-only.

[Text fragments article] Rename textStartstart and textEndend#9815

Merged
tomayac merged 1 commit intomainfrom
tomayac-patch-6
Mar 27, 2023
Merged

[Text fragments article] Rename textStartstart and textEndend#9815
tomayac merged 1 commit intomainfrom
tomayac-patch-6

Conversation

@tomayac
Copy link
Copy Markdown
Member

@tomayac tomayac commented Mar 27, 2023

Changes proposed in this pull request:

@tomayac tomayac requested a review from rachelandrew March 27, 2023 08:00
@netlify
Copy link
Copy Markdown

netlify bot commented Mar 27, 2023

Deploy Preview for web-dev-staging ready!

Name Link
🔨 Latest commit b674b42
🔍 Latest deploy log https://app.netlify.com/sites/web-dev-staging/deploys/64214d2a02ce1b0008f2360e
😎 Deploy Preview https://deploy-preview-9815--web-dev-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

Copy link
Copy Markdown
Collaborator

@rachelandrew rachelandrew left a comment

Choose a reason for hiding this comment

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

We should move this to d.c.c.

@tomayac tomayac merged commit 79eb8e6 into main Mar 27, 2023
@tomayac tomayac deleted the tomayac-patch-6 branch March 27, 2023 12:02
@tomayac
Copy link
Copy Markdown
Member Author

tomayac commented Mar 27, 2023

We should move this to d.c.c.

It's supported by Chrome and Safari and Firefox is positive on it.

@rachelandrew
Copy link
Copy Markdown
Collaborator

it's just had a spec change though?

@tomayac
Copy link
Copy Markdown
Member Author

tomayac commented Mar 27, 2023

This is purely a housekeeping renaming. It's not web exposed (see GoogleChromeLabs/text-fragments-polyfill#148 (comment)) for one tiny bit of non-defined exposition.

1 similar comment
@tomayac
Copy link
Copy Markdown
Member Author

tomayac commented Mar 27, 2023

This is purely a housekeeping renaming. It's not web exposed (see GoogleChromeLabs/text-fragments-polyfill#148 (comment)) for one tiny bit of non-defined exposition.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants