ucdavis / ucdavis/ipa-client-angular
Courses table bottom totals should reflect visible rows
Open
Nobody has claimed this yet.
enhancement
- Dominant language
- JavaScript
- Stars
- 2
- Forks
- 3
- PR merge metrics
- No merged PRs in 30d
Description
Currently, the seat sum for a term will always sum all rows, even ones not visible thanks to a filter.
It should really reflect the visible total only, to make it more useful.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the Courses table and the code that calculates the seat sum, then trace how filtering determines which rows are visible. Update the total so it uses only visible rows, and verify that filtered and unfiltered tables display the expected totals.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- angularjs, javascript
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100