Skip to content

feat: reintroducing table cell schema changes [TOL-2249] #622

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.

Sign up for GitHub

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

Open
Aodhagan Murphy (aodhagan-cf) wants to merge 14 commits into
base: master
Choose a base branch
from
Open

feat: reintroducing table cell schema changes [TOL-2249] #622

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
c6f2380
feat: update schema for rich text types
2wce Jul 17, 2024
6617c21
chore: update snapshots
2wce Jul 17, 2024
36cced0
chore: update schema
2wce Jul 17, 2024
a366d2c
chore: update snapshots
2wce Jul 17, 2024
73eb553
chore: update table cell nodetype
2wce Jul 17, 2024
b3f4bd2
feat: adjusting table cell/header schemas [TOL-2249]
aodhagan-cf Jul 19, 2024
0cacfdf
feat: adding comment to explain reason for coupling [TOL-2249]
aodhagan-cf Jul 19, 2024
e4eeae2
chore: adjusting table schema [TOL-2249]
aodhagan-cf Jul 19, 2024
3d02a7d
chore: updating snapshot tests [TOL-2249]
aodhagan-cf Jul 19, 2024
7b248ab
chore: updating comment [TOL-2249]
aodhagan-cf Jul 19, 2024
e5ea016
chore: adjusting table blocks schemas to follow list block pattern [T…
aodhagan-cf Jul 21, 2024
12c40df
chore: adding same rules as list item for table cell [TOL-2249]
aodhagan-cf Jul 23, 2024
6b3ea05
chore: updating snapshots [TOL-2249]
aodhagan-cf Jul 23, 2024
3ec95ae
Merge branch 'master' into feat/TOL-2249
aodhagan-cf Jul 24, 2024
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
2 changes: 1 addition & 1 deletion packages/rich-text-types/src/schemaConstraints.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ export type ListItemBlockEnum =

export type TableRowBlockEnum = BLOCKS.TABLE_CELL | BLOCKS.TABLE_HEADER_CELL;

export type TableCellEnum = BLOCKS.PARAGRAPH | BLOCKS.UL_LIST | BLOCKS.OL_LIST;
export type TableCellEnum = ListItemBlockEnum;
Copy link
Member

@z0al z0al Jul 23, 2024

Choose a reason for hiding this comment

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

  • Nit: Can we have a dedicated enum here? it would be handy when adjusting the normalisation logic in the editor
  • You also need to adjust the entry for table cell in the CONTAINERS below


export type TableHeaderCellEnum = BLOCKS.PARAGRAPH;

Expand Down
11 changes: 11 additions & 0 deletions packages/rich-text-types/src/schemas/generated/table-cell.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -72,6 +72,17 @@
},
"TableCellEnum": {
"enum": [
"blockquote",
"embedded-asset-block",
"embedded-entry-block",
"embedded-resource-block",
"heading-1",
"heading-2",
"heading-3",
"heading-4",
"heading-5",
"heading-6",
"hr",
"ordered-list",
"paragraph",
"unordered-list"
Expand Down