Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Resolve Errors] [Review Warnings] Tables - full borders will only show when overflowing (horizontal scrolling active) #583

Merged
merged 3 commits into from
May 28, 2024

Conversation

shindigira
Copy link
Contributor

@shindigira shindigira commented May 25, 2024

closes #547

Note: Requires cfpb/design-system-react#358 to be merged in first

Changes

  • enhancement-style(tables): full borders will only show for overflowing
  • chore(tables): removed the need for showTableBorders

How to test this PR

  1. yarn install to get the latest DSR update with the table ref change
  2. use all-errors-warnings-no-syntax-errors.csv or any register with multifield errors that causes overflowing/horizontal scrolling
  3. Verify that tables without overflowing don't have full borders, tables that do have overflowing/horizontal scrolling have full borders

Screenshot

Screenshot 2024-05-25 at 11 47 57 AM

…g tables, removed the need for showTableBorders
@shindigira shindigira changed the title enhancement-style(tables): full borders will only show for overflowing [Resolve Errors] [Review Warnings] Tables - full borders will only show for overflowing May 25, 2024
@shindigira shindigira changed the title [Resolve Errors] [Review Warnings] Tables - full borders will only show for overflowing [Resolve Errors] [Review Warnings] Tables - full borders will only show when overflowing (horizontal scrolling active) May 27, 2024
Copy link
Contributor

@billhimmelsbach billhimmelsbach left a comment

Choose a reason for hiding this comment

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

Looking good! One quick perf question.

src/pages/Filing/FilingApp/FieldEntry.tsx Show resolved Hide resolved
Copy link
Contributor

@billhimmelsbach billhimmelsbach left a comment

Choose a reason for hiding this comment

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

Sounds good! Looking forward to the follow-up PR!

@shindigira shindigira merged commit 49ba829 into main May 28, 2024
4 checks passed
@billhimmelsbach billhimmelsbach deleted the 547-table-borders branch May 31, 2024 14:44
@natalia-fitzgerald
Copy link

natalia-fitzgerald commented Jun 4, 2024

@shindigira
In the screenshot the bottom border is missing. It should be there if the other borders are present.

I noticed that on AWS sometimes the bottom border renders and other times it does not.

Bottom border renders
Screenshot 2024-06-03 at 10 57 31 PM

Bottom border does not render
Screenshot 2024-06-03 at 10 50 48 PM

Screenshot 2024-06-03 at 10 54 32 PM

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.

[Resolve Errors] [Review Warnings] Tables - selectively enable borders based on if the table is overflowing
3 participants