Reorganize frontend events and simplify view binding documentation - #128
Merged
Conversation
… parameter - nested_views: the per-view model separation is gone (all _bind data goes to one root model) and _bind's view parameter is now an inert no-op; rewrite the routing section to choose the target view via the matching ..._view_model_update() call and drop the no-op view= from the examples - frontend: refresh the stale cs_event listing and document the whitelisted control_by_id/control_global/binding_call calls plus the new view importing parameter on _event_client - follow_up_action: document the whitelisted control calls and the view parameter - xml_templating: drop a leftover duplicate _bind reference from the _bind_edit sweep Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NjYGeMm8yvpJhavp6hdGyN
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR reorganizes the frontend events documentation with improved categorization and adds comprehensive documentation for control method invocation. It also simplifies the view binding API by removing the now-obsolete
viewparameter from_bind()calls and clarifying the single unified model architecture.Key Changes
Documentation Reorganization
Frontend events (
docs/cookbook/event_navigation/frontend.md):control_by_id,control_global, andbinding_calleventsviewparameter for scoping control lookups to specific views (popup, popover, nested, etc.)viewparameter moving fromt_argto a dedicated importing parameterNested views (
docs/cookbook/view/nested_views.md):viewparameter from allclient->_bind()calls (now defaults to main view)viewparameter is now inert and kept only for backward compatibilityFollow-up actions (
docs/cookbook/expert_more/follow_up_action.md):follow_up_action()viewparameter for scoping control lookups in follow-up actionsXML templating (
docs/cookbook/view/xml_templating.md):Notable Implementation Details
viewparameter to_bind()continues to workview_model_update(),nest_view_model_update(), etc.) control which rendered view receives updates`` to maintain argument positionshttps://claude.ai/code/session_01NjYGeMm8yvpJhavp6hdGyN