Skip to content

OCD-5296 | Improve the styling and layout of embedded report iframes - #1520

Open
mattystank wants to merge 3 commits into
chpladmin:stagingfrom
mattystank:OCD-5296
Open

OCD-5296 | Improve the styling and layout of embedded report iframes#1520
mattystank wants to merge 3 commits into
chpladmin:stagingfrom
mattystank:OCD-5296

Conversation

@mattystank

Copy link
Copy Markdown
Contributor

This pull request updates the ChplCharts page to improve the styling and layout of embedded report iframes. The main changes involve introducing custom styles for the CardContent and iframe elements to ensure better visual presentation and consistency.

Styling improvements:

  • Added a new reportCardContent style to makeStyles to remove default padding, hide overflow, and eliminate extra bottom padding from the last child, ensuring the embedded report fills the card cleanly.
  • Added a new iframe style to makeStyles to remove borders, set width to 100%, display as a block, and adjust the bottom margin for better alignment within the card.

Component usage updates:

  • Applied the new reportCardContent style to CardContent and the new iframe style to the embedded iframe element, replacing inline and deprecated attributes for improved consistency and maintainability.[#OCD-5296]

@mattystank
mattystank marked this pull request as ready for review July 14, 2026 19:08
@mattystank
mattystank requested a review from andlar as a code owner July 14, 2026 19:08
Comment thread src/app/pages/charts/charts.jsx
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants