Skip to content

Conversation

@Anca2022
Copy link
Contributor

@Anca2022 Anca2022 commented Dec 17, 2025

PR Checklist

  • Linked issue added (e.g., Fixes #123)
  • If styles were updated:
    • Stylesheet version has been bumped

Summary

  • create Footnote component
  • added Footer notes content type in Drupal interface - added the following disclaimer
image
  • create .policy-table class to set a minimum width of 600px and horizontal scrolling on smaller screens

(INTORG-220)

@Anca2022 Anca2022 self-assigned this Dec 17, 2025
@Anca2022
Copy link
Contributor Author

image

@Anca2022 Anca2022 marked this pull request as ready for review December 17, 2025 14:37
@Anca2022
Copy link
Contributor Author

Tables on the Policy page contain a large amount of structured data, which makes them difficult to read when transformed into the mobile “card” layout used elsewhere in the site.

To improve readability, this PR uses .policy-table to set a minimum width of 600px and horizontal scrolling on smaller screens.

image

@Anca2022 Anca2022 changed the title chore: create footnotes component chore: support Policy Briefs pages (footnotes + table layout) Dec 18, 2025
@Anca2022
Copy link
Contributor Author

image

@@ -0,0 +1,24 @@
.footer-notes {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: The CSS file is named footnote.css but uses .footer-notes as the class. Consider aligning naming (footnote vs footer-notes) for clarity.

white-space: normal;
}

.table-wrapper:has(table.policy-table) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just a suggestion, for horizontal scroll on mobile, consider adding a visual indicator that the table is scrollable (like a shadow fade on the right edge of the table or make the scrollbar persistent)

@Anca2022 Anca2022 merged commit bde9d11 into main Jan 15, 2026
@Anca2022 Anca2022 deleted the am/policy-briefs branch January 15, 2026 07:55
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.

3 participants