-
Notifications
You must be signed in to change notification settings - Fork 1.7k
adding row selection mode #6090
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.
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
base: main
Are you sure you want to change the base?
adding row selection mode #6090
Conversation
CodSpeed Performance ReportMerging this PR will not alter performanceComparing Summary
Footnotes |
Greptile OverviewGreptile SummaryAdded
Confidence Score: 5/5
Sequence DiagramsequenceDiagram
participant User
participant DataEditor
participant GlideDataGrid
User->>DataEditor: Set row_selection_mode prop
Note over DataEditor: row_selection_mode: "auto" | "multi"
DataEditor->>GlideDataGrid: Pass row_selection_mode to underlying library
alt row_selection_mode = "auto"
User->>GlideDataGrid: Click row marker
GlideDataGrid->>GlideDataGrid: Adapt behavior to touch/mouse
Note over GlideDataGrid: Single select on click,<br/>multi-select with Ctrl
else row_selection_mode = "multi"
User->>GlideDataGrid: Click row marker
GlideDataGrid->>GlideDataGrid: Act as if Ctrl is pressed
Note over GlideDataGrid: Always multi-select
end
GlideDataGrid->>DataEditor: Update selection state
DataEditor->>User: Reflect updated row selection
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No files reviewed, no comments
|
can you run pre-commit to update the pyi_hashes.json file |
Done :) |
In order to control the row selection behavior of the editable grid, we need to have this prop, this changes behavior from the default multi select by holding control, to multi select with mouse click.
All Submissions:
Type of change
Please delete options that are not relevant.
New Feature Submission: