codeforboston / codeforboston/windfall-elimination
Support styled print page
- Dominant language
- JavaScript
- Stars
- 23
- Forks
- 44
- PR merge metrics
- No merged PRs in 30d
Description
Currently, the print component sets `innerHTML` directly before triggering a print, which can be dangerous in React since we're taking control away from it. This might also be why the print page doesn't have the styles of the rest of the app. We might be able to improve this by using CSS to hint to the browser not to print certain parts of the page (like the sidebar and header), which would allow us to print the body of the page which styles!
https://github.com/codeforboston/windfall-elimination/blob/develop/src/pages/print.tsx#L221
Contributor guide
No contributing guide indexed for this repository
Research direction
Start in src/pages/print.tsx around line 221, where the print component sets innerHTML before triggering printing. Review how the page is rendered and how styles are applied, then compare the proposed CSS approach for excluding the sidebar and header. Done means the print page retains the app’s relevant styles without taking control away from React.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- react, typescript
- Domain
- frontend
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100