Closed GeorgeGoodall-GovUk closed 3 weeks ago
The changes in this pull request introduce new CSS classes and styles to enhance the layout and appearance of tables and scrollable elements in the application. Specifically, new classes such as .dl-scrollable
and .dl-table
are added to the SCSS file, defining properties for horizontal and vertical scrolling. The HTML structure of the table component is also modified to incorporate these classes, along with updates to the logic for rendering cell content, improving the overall functionality and styling of the tables.
File | Change Summary |
---|---|
src/assets/scss/index.scss | Added new classes: .dl-scrollable , .dl-table , .dl-table__head , .dl-table__header , .dl-table__cell--error . |
src/views/components/table.html | Updated class attributes for <div> and <table> ; modified <th> elements; introduced cellText for cell content handling. |
Objective | Addressed | Explanation |
---|---|---|
Update table with new styles (#598) | β |
In a world of tables, neat and bright,
New styles emerge, a splendid sight.
Scrollable wonders, headers that gleam,
With every change, we build our dream!
Hoppy times await, letβs take a leap,
For in our code, the magic runs deep! πβ¨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Status | Category | Percentage | Covered / Total |
---|---|---|---|
π΅ | Lines | 62.3% | 3539 / 5680 |
π΅ | Statements | 62.3% | 3539 / 5680 |
π΅ | Functions | 65.5% | 150 / 229 |
π΅ | Branches | 81.08% | 420 / 518 |
What type of PR is this? (check all applicable)
Description
Updates the style of the table, including making the table heading sticky
Related Tickets & Documents
QA Instructions, Screenshots, Recordings
view any page with a table and ensure the style's are correct
Added/updated tests?
No
Summary by CodeRabbit
New Features
Improvements