admin/HighCostDrugsDemo
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

docs: update progress.txt with iteration 21 (Task E.5 complete)

Browse Source
This commit is contained in:
Andrew Charlwood
2026-02-07 22:35:20 +00:00
parent 901bf39dda
commit 4f9145c461
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
progress.txt
+1 -1
View File
@@ -855,7 +855,7 @@ Working Dash application with 2 views (Patient Pathways + Trust Comparison), 13
- `dash_app/components/chart_card.py` — renamed heatmap metric toggle labels - `dash_app/components/chart_card.py` — renamed heatmap metric toggle labels
- `dash_app/components/trust_comparison.py` — renamed TC heatmap metric toggle labels - `dash_app/components/trust_comparison.py` — renamed TC heatmap metric toggle labels
- `IMPLEMENTATION_PLAN.md` — marked E.5 subtasks [x], all completion criteria [x] - `IMPLEMENTATION_PLAN.md` — marked E.5 subtasks [x], all completion criteria [x]
### Committed: [pending] ### Committed: 901bf39 "fix: responsive chart heights + rename cost metric labels (Task E.5)"
### Patterns discovered: ### Patterns discovered:
- Plotly `responsive=True` on `dcc.Graph` + `autosize=True` in figure layout is insufficient on its own — the Plotly-generated inner divs (`.js-plotly-plot`, `.plot-container`, `.svg-container`) need explicit `height: 100%` CSS to inherit the flex container height. Without this, they default to their content height. - Plotly `responsive=True` on `dcc.Graph` + `autosize=True` in figure layout is insufficient on its own — the Plotly-generated inner divs (`.js-plotly-plot`, `.plot-container`, `.svg-container`) need explicit `height: 100%` CSS to inherit the flex container height. Without this, they default to their content height.
- The `!important` on the height rule is needed because Plotly sets inline styles on these elements. - The `!important` on the height rule is needed because Plotly sets inline styles on these elements.