Repository navigation
styles: use fixed widths for user roles table in large screens and allow wrap text in page header - #254
styles: use fixed widths for user roles table in large screens and allow wrap text in page header #254dcoa wants to merge 5 commits into
Conversation
The column widths, fixed layout and row padding were nested under `&.team-members-table`, which compiles to `.authz-module.team-members-table`. No element carried that class, so none of it applied and the columns resized to each page's content. Match the class as a descendant of `.authz-module` and set it on the team members DataTable. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
openedx-webhooks
added
open-source-contribution
|
Thanks for the pull request, @dcoa! This repository is currently maintained by Once you've gone through the following steps feel free to tag them in a comment and let them know that your changes are ready for engineering review. 🔘 Get product approvalIf you haven't already, check this list to see if your contribution needs to go through the product review process.
🔘 Provide contextTo help your reviewers and other members of the community understand the purpose and larger context of your changes, feel free to add as much of the following information to the PR description as you can:
🔘 Get a green buildIf one or more checks are failing, continue working on your changes until this is no longer the case and your build turns green. DetailsWhere can I find more information?If you'd like to get more details on all aspects of the review process for open source pull requests (OSPRs), check out the following resources: When can I expect my changes to be merged?Our goal is to get community contributions seen and reviewed as efficiently as possible. However, the amount of time that it takes to review and merge a PR can vary significantly based on factors such as:
💡 As a result it may take up to several weeks or months to complete a review and merge your PR. |
…actions A username or email without spaces could not wrap, so on the audit user page a long email overflowed its column and slid under the Assign Role button. Let the title and subtitle break inside their column. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
bea4b4d to
389f717
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #254 +/- ##
=======================================
Coverage 98.71% 98.71%
=======================================
Files 80 80
Lines 1396 1396
Branches 438 438
=======================================
Hits 1378 1378
Misses 18 18 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
@jacobo-dominguez-wgu I was thinking the table might have a minimum width so the relevant information is properly displayed in small screens, This is out of the scope as a proposal but wanted to discuss. For example, for the team members a This is how the table scrolls today Screencast.from.2026-10-09.10-16-28.webmThis is how it looks with the minimum width Screencast.from.2026-10-09.10-20-05.webm |
dcoa
changed the title
I agree, looks better with it. |
3 participants
Description
The columns has a fix width based on the Figma design and the values where implemented in #222 but there were not applied as you can see here #222 (review). The PR enable the table static width from medium screens, in small screens it keep the default auto-resizing behavior for paragon.
Screencast.from.2026-10-09.09-40-34.webm
It also works in a edge case in the header where a title and subtitle are long they don't wrap instead overflow the page.
Before
After