Skip to content

Update serialization docs to reflect support for workspace comments #10266

Description

@zspriggs

Location

https://docs.blockly.com/guides/configure/serialization/#serializer-hooks

Type

  • Text
  • Image or Gif
  • Other

Suggested content

The "Save and Load" doc serialization.mdx claims that workspace comments are not supported by JSON serialization, but support for this was added in #7927

The stale doc should be updated to remove this claim. The following serializer hooks example should also be renamed to something more applicable, since it uses workspace-comments as an example which could be confusing.

Additional context

No response

Activity

  1. added theissue type on Aug 12, 2026
  2. added
    good first issueIssues that are good candidates for your first contribution to Blockly
    help wantedExternal contributions actively solicited
    on Aug 13, 2026
  3. zspriggs commented on Aug 13, 2026

    @zspriggs
    ContributorAuthor

    Bonus: this page also includes a broken link on the text "built-in priorities". The correct link is here. This link fix can also be included on a PR that fixes this issue.

  4. Johnkothapalli commented on Aug 13, 2026

    @Johnkothapalli

    I’m working on this. I’ll keep it to the documentation scope described here: document built-in workspace-comment JSON serialization, rename the custom serializer example so it no longer implies comments need a plugin, fix the priorities link, and validate the docs lint/format/build.

  5. moved this from Todo to In Progress in Blocklyon Aug 13, 2026
  6. ritfiz commented on Oct 24, 2026

    @ritfiz
    Contributor

    @zspriggs - I have added a PR (#10628) that does the following

    • Show from docs that workspace-comment is supported out of the box
    • Added a custom myPlugin in the example
    • Updated the example on the clear stage to show what happens to the variables of the myPlugin serializer.

    One minor thing I noticed in the part of "When a serializer is triggered, two things happen" -
    It should mention that the clear and load does not happen for the triggered serializer, but, clear happens for all in the priority order of smallest to largest, and then load happens in the reverse priority order.

  7. moved this from In Progress to Done in Blocklyon Oct 26, 2026
  8. added a commit that references this issue on Oct 26, 2026
    a8f3dd2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    area: docs & examplesgood first issueIssues that are good candidates for your first contribution to Blocklyhelp wantedExternal contributions actively solicited

    Type

    Projects

    • Status
      Done

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions