rstaib/jquery-bootgrid

getSelectedRows - Works with Toggle All - Sort of

Open

#128 opened on Apr 17, 2015

View on GitHub
 (4 comments) (0 reactions) (1 assignee)JavaScript (976 stars) (375 forks)batch import
help wanted

Description

I've been trying getSelectedRows on 1.2.0rc and I'm running into same problem I had when I was trying to "hack" 1.1.4

I can get an array of all rows if I use the heading toggle checkbox to check all rows. But if I check/uncheck individual rows, it does not affect the output. Seems like .selectedRows is not affected by individual row changes?

I'm setting (and I think I've tried all combinations): selection: true, multiSelect: true, rowSelect: true, keepSelection: true,

The same thing occurs when using .on("selected.rs.jquery.bootgrid"

Any ideas how to fix this?

Otherwise, excellent plug in. Everything else is working great. Like the templates and formatters!

Thanks Scott

Contributor guide

Tech stack
javascriptcsshtml
Domain
frontend
Issue type
bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
None
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
50
Research direction
The issue describes that getSelectedRows only returns rows selected via the toggle all checkbox, not individual row selections. The fix likely involves updating the internal selection array when individual checkboxes are toggled. Examine the plugin's source code for selection handling, particularly methods like 'select' and 'deselect', and ensure that individual row selection updates the selectedRows collection. The comments in the issue may provide additional context.