Skip to content

Complete the update to 47 guide#19900

Merged
gorzelinski merged 11 commits intostablefrom
ck/19826-update-47
Mar 10, 2026
Merged

Complete the update to 47 guide#19900
gorzelinski merged 11 commits intostablefrom
ck/19826-update-47

Conversation

@gorzelinski
Copy link
Copy Markdown
Contributor

🚀 Summary

This PR adds the info about the new ordered list type.


📌 Related issues


💡 Additional information

Optional: Notes on decisions, edge cases, or anything helpful for reviewers.


🧾 Checklists

Use the following checklists to ensure important areas were not overlooked.
This does not apply to feature-branch merges.
If an item is not relevant to this type of change, simply leave it unchecked.

Author checklist

  • Is the changelog entry intentionally omitted?
  • Is the change backward-compatible?
  • Have you considered the impact on different editor setups and core interactions? (e.g., classic/inline/multi-root/many editors, typing, selection, paste, tables, lists, images, collaboration, pagination)
  • Has the change been manually verified in the relevant setups?
  • Does this change affect any of the above?
  • Is performance impacted?
  • Is accessibility affected?
  • Have tests been added that fail without this change (against regression)?
  • Have the API documentation, guides, feature digest, and related feature sections been updated where needed?
  • Have metadata files (ckeditor5-metadata.json) been updated if needed?
  • Are there any changes the team should be informed about (e.g. architectural, difficult to revert in future versions or having impact on other features)?
  • Were these changes documented (in Logbook)?

Reviewer checklist

  • PR description explains the changes and the chosen approach (especially when performance, API, or UX is affected).
  • The changelog entry is clear, user‑ or integrator-facing, and it describes any breaking changes.
  • All new external dependencies have been approved and mentioned in LICENSE.md (if any).
  • All human-readable, translateable strings in this PR been introduced using t() (if any).
  • I manually verified the change (e.g., in manual tests or documentation).
  • The target branch is correct.

pszczesniak
pszczesniak previously approved these changes Mar 9, 2026
Copy link
Copy Markdown
Contributor

@pszczesniak pszczesniak left a comment

Choose a reason for hiding this comment

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

Reviewed the areas I was involved in - LGTM 👍

Mati365
Mati365 previously approved these changes Mar 9, 2026
@Witoso
Copy link
Copy Markdown
Member

Witoso commented Mar 9, 2026

hey, usually update guide has almost all the release highlights? Do you plan to change the process?

@gorzelinski gorzelinski dismissed stale reviews from Mati365 and pszczesniak via ba8aacc March 9, 2026 11:29
@gorzelinski
Copy link
Copy Markdown
Contributor Author

gorzelinski commented Mar 9, 2026

AFAIK, due to the lack of Bartek, teams are mostly responsible for particular docs. But I added those missing entries. You can check it out in your spare time. cc @Witoso

@gorzelinski gorzelinski requested a review from Witoso March 9, 2026 11:33
@gorzelinski gorzelinski mentioned this pull request Mar 10, 2026
18 tasks
@Witoso
Copy link
Copy Markdown
Member

Witoso commented Mar 10, 2026

One comment, otherwise ok 👍

@gorzelinski gorzelinski requested review from Mati365 and Witoso March 10, 2026 07:38
Mati365
Mati365 previously approved these changes Mar 10, 2026
@gorzelinski gorzelinski merged commit 870bc4a into stable Mar 10, 2026
10 checks passed
@gorzelinski gorzelinski deleted the ck/19826-update-47 branch March 10, 2026 07:52
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