Conversation
Reviewer's GuideEnsures database-linked views open with correct metadata and navigation behavior, updates mention page interactions to always navigate rather than open a modal, and tightens link/mention rendering to avoid empty href content and unintended pointer events. Sequence diagram for clicking a page mention in the editorsequenceDiagram
actor User
participant Editor as SlateEditor
participant MentionPage
participant Navigator as NavigationService
User->>Editor: Click mention inline element
Editor->>MentionPage: onClick event handler
MentionPage->>MentionPage: Check noAccess
alt noAccess is true
MentionPage-->>Editor: Ignore click
else Same page reference
MentionPage->>MentionPage: handleScrollToBlock()
MentionPage->>Editor: Scroll to target block
else Different page reference
MentionPage->>Navigator: navigateToView(pageId, blockId) (deferred with timeout)
Navigator-->>User: Display target view
end
Class diagram for updated view and mention componentsclassDiagram
class ViewModal {
+string workspaceId
+string viewId
+boolean open
+function onClose()
+View fallbackMeta
+View resolvedView
+ViewMeta viewMeta
+function loadView(viewId)
+function loadPageDoc(viewId)
}
class View {
+string view_id
+string name
+ViewLayout layout
+Icon icon
+ViewExtra extra
}
class ViewExtra {
+Cover cover
}
class ViewMeta {
+string name
+Icon icon
+Cover cover
+ViewLayout layout
+string viewId
+string[] visibleViewIds
+ViewExtra extra
+string workspaceId
}
class MentionPage {
+string pageId
+string blockId
+MentionType type
+boolean noAccess
+View meta
+string content
+function handleScrollToBlock()
}
class Leaf {
+LeafProps leaf
+TextNode text
+function Leaf(props)
}
class MentionLeaf {
+Mention mention
+TextNode text
+function MentionLeaf(mention, text, children)
}
ViewModal --> View : uses as fallbackMeta
ViewModal --> ViewMeta : builds
View --> ViewExtra : has
ViewExtra --> Cover : has
MentionPage --> View : uses as meta
Leaf --> MentionLeaf : may render
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
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.
Description
Checklist
General
Testing
Feature-Specific
Summary by Sourcery
Improve how views and mentioned pages are resolved and opened from documents, especially for database and same-page references.
Bug Fixes:
Enhancements: