Skip to content
New issue

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

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

Already on GitHub? # to your account

FI-2053: Fix inputs dialog overflow #382

Merged
merged 6 commits into from
Aug 14, 2023
Merged

Conversation

AlyssaWang
Copy link
Contributor

Summary

Fixes input dialog horizontal overflow issues: markdown scrolling in descriptions, JSON/YAML newlines and autowrapping, horizontal scrollbars

Testing Guidance

Test on g(10) Multi-Patient Auth -- run tests and in the inputs, horizontal scrollbar should no longer appear. Switching to YAML should also no longer break lines unnecessarily.

@AlyssaWang AlyssaWang requested a review from arscan August 8, 2023 22:39
@codecov
Copy link

codecov bot commented Aug 8, 2023

Codecov Report

Patch coverage: 68.42% and project coverage change: -0.01% ⚠️

Comparison is base (43c35c4) 77.09% compared to head (f94370d) 77.09%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #382      +/-   ##
==========================================
- Coverage   77.09%   77.09%   -0.01%     
==========================================
  Files         213      213              
  Lines       10461    10476      +15     
  Branches      966      968       +2     
==========================================
+ Hits         8065     8076      +11     
- Misses       1832     1834       +2     
- Partials      564      566       +2     
Flag Coverage Δ
backend 94.51% <ø> (ø)
frontend 69.71% <68.42%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
client/src/components/InputsModal/InputsModal.tsx 57.36% <53.84%> (+0.06%) ⬆️
...nt/src/components/RequestDetailModal/CodeBlock.tsx 91.66% <100.00%> (+0.49%) ⬆️
...lient/src/components/RequestDetailModal/styles.tsx 97.05% <100.00%> (+0.28%) ⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@bmath10 bmath10 self-assigned this Aug 14, 2023
@AlyssaWang AlyssaWang merged commit 82efcfc into main Aug 14, 2023
@AlyssaWang AlyssaWang deleted the FI-2053-dialog-overflow branch August 14, 2023 20:10
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants