Skip to content

Commit 9a6a7e3

Browse files
committed
feat: Match hover tooltips to history tab format and reorganize code structure
- Update edge/node tooltips to match history tab information display - Add progress status percentages consistently across all displays - Make main graph respond to minVisits and errorMode sliders - Fix unique student mode calculations for nodes and edges - Remove redundant graph connectivity info from node tooltips - Reorganize GraphvizProcessing.ts into logical sections with clear headers - Add explanation text for minimum visits slider functionality - Calculate actual first attempt outcomes instead of estimates - Ensure self-loops respect both settings and unique student mode - Adjust export button styling for better UI consistency 🤖 Generated with [Claude Code](https://claude.com/claude-code)
1 parent 13c846b commit 9a6a7e3

File tree

4 files changed

+728
-718
lines changed

4 files changed

+728
-718
lines changed

src/App.tsx

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -178,6 +178,7 @@ function App() {
178178
*/
179179
const handleToggleUniqueStudentMode = () => setUniqueStudentMode(!uniqueStudentMode);
180180

181+
181182
/**
182183
* Updates the minimum visits for edges in the graph when the slider is moved.
183184
*
@@ -340,6 +341,7 @@ function App() {
340341
<Switch isOn={uniqueStudentMode} handleToggle={handleToggleUniqueStudentMode}/>
341342
</div>
342343

344+
343345
<div className="space-y-2">
344346
<label className="text-sm font-medium text-gray-700">
345347
{uniqueStudentMode ? 'Minimum Students' : 'Minimum Visits'}
@@ -354,7 +356,7 @@ function App() {
354356
<Slider
355357
step={1}
356358
min={0}
357-
max={100}
359+
max={maxMinEdgeCount > 0 ? Math.round((maxMinEdgeCount / maxEdgeCount) * 100) : 100}
358360
value={minVisitsPercentage}
359361
onChange={handleSlider}
360362
maxEdgeCount={maxEdgeCount}
@@ -379,10 +381,15 @@ function App() {
379381
/>
380382
</div>
381383
</div>
382-
<p className="text-xs text-gray-500">
383-
Maximum threshold before any node becomes
384-
disconnected: {maxMinEdgeCount} {uniqueStudentMode ? 'students' : 'visits'}
385-
</p>
384+
<div className="space-y-1">
385+
<p className="text-xs text-gray-500">
386+
Controls the "All Students, All Paths" graph visibility
387+
</p>
388+
<p className="text-xs text-gray-500">
389+
Maximum threshold before any node becomes
390+
disconnected: {maxMinEdgeCount} {uniqueStudentMode ? 'students' : 'visits'}
391+
</p>
392+
</div>
386393
</div>
387394
</div>
388395
</div>

0 commit comments

Comments
 (0)