Skip to content

Conversation

@stanislaw
Copy link
Collaborator

@stanislaw stanislaw commented Jul 20, 2025

This change updates the Grammar Element Form to include the new properties IS_COMPOSITE, PREFIX, and VIEW_STYLE as hidden fields. This ensures that saving an element preserves these fields in the resulting SDoc file. In the future, these hidden fields will be converted into regular string/checkbox inputs that users can edit.

This change also removes a previously implemented “clever” optimization that skipped saving an element if none of its fields appeared to have changed. Over time, this check became outdated due to the introduction of new features such as HUMAN_TITLE, PROPERTIES/IS_COMPOSITE, PROPERTIES/VIEW_STYLE, and others.

Closes #2341

@stanislaw stanislaw added this to the 2025-Q3 milestone Jul 20, 2025
@stanislaw stanislaw force-pushed the stanislaw/new_section_included_documents branch from f7677b0 to 79d14e5 Compare July 23, 2025 20:38
…TITLE fields

This change updates the Grammar Element Form to include the new properties IS_COMPOSITE, PREFIX, and VIEW_STYLE as hidden fields. This ensures that saving an element preserves these fields in the resulting SDoc file. In the future, these hidden fields will be converted into regular string/checkbox inputs that users can edit.

This change also removes a previously implemented “clever” optimization that skipped saving an element if none of its fields appeared to have changed. Over time, this check became outdated due to the introduction of new features such as HUMAN_TITLE, PROPERTIES/IS_COMPOSITE, PROPERTIES/VIEW_STYLE, and others.

Related to: #2193

Closes: #2341
@stanislaw stanislaw force-pushed the stanislaw/new_section_included_documents branch from 79d14e5 to fbdec7a Compare July 23, 2025 20:46
@stanislaw stanislaw changed the title server: fix updating HUMAN_TITLE in Edit Grammar Element UI UI: Edit Grammar Element: support updating composite nodes and HUMAN_TITLE fields Jul 23, 2025
@stanislaw stanislaw merged commit acb81ed into main Jul 24, 2025
25 checks passed
@stanislaw stanislaw deleted the stanislaw/new_section_included_documents branch July 24, 2025 07:44
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.

Bug: No-Op FIELD HUMAN NAME in Edit Grammar Element UI

2 participants