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

fix(app): fix small bugs on ODD #13618

Merged
merged 1 commit into from
Sep 21, 2023
Merged

Conversation

koji
Copy link
Contributor

@koji koji commented Sep 21, 2023

Overview

Fix failed run modal text wrapping issue and RunSummary splash screen's textTransform issue.

before
Screenshot 2023-09-21 at 11 56 08 AM

after
Screenshot 2023-09-21 at 12 05 41 PM

close RAUT-703 and RAUT-704

Test Plan

Changelog

Review requests

Risk assessment

low

fix failed run modal text wrapping issue and RunSummary splash screen's textTransform issue

close RAUT-703 and RAUT-704
@koji koji requested a review from a team September 21, 2023 16:11
@codecov
Copy link

codecov bot commented Sep 21, 2023

Codecov Report

Merging #13618 (06a299d) into chore_release-7.0.0 (1ac718e) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@                 Coverage Diff                  @@
##           chore_release-7.0.0   #13618   +/-   ##
====================================================
  Coverage                71.30%   71.30%           
====================================================
  Files                     2425     2425           
  Lines                    68193    68193           
  Branches                  7949     7949           
====================================================
  Hits                     48628    48628           
  Misses                   17703    17703           
  Partials                  1862     1862           
Flag Coverage Δ
app 68.91% <ø> (ø)

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

Files Changed Coverage Δ
...OnDeviceDisplay/RunningProtocol/RunFailedModal.tsx 84.78% <ø> (ø)
app/src/pages/OnDeviceDisplay/RunSummary.tsx 11.66% <ø> (ø)

Comment on lines +105 to +107
css={css`
word-break: break-word;
`}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Using css since wordBreak="break-word" doesn't work.
I think at some point we would need to update StyleProps to add new css properties.
I guess we will be able to work on this when we start working on the design system unification.

Copy link
Contributor

Choose a reason for hiding this comment

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

Totally agree. It would make things a bit easier.

@koji koji marked this pull request as ready for review September 21, 2023 16:13
@koji koji requested a review from a team as a code owner September 21, 2023 16:13
@koji koji requested review from shlokamin and removed request for a team September 21, 2023 16:13
Copy link
Contributor

@mjhuff mjhuff left a comment

Choose a reason for hiding this comment

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

lgtm, thank you!

@koji koji removed the request for review from shlokamin September 21, 2023 16:23
@koji koji merged commit 3633c12 into chore_release-7.0.0 Sep 21, 2023
@koji koji deleted the fix_small-odd-ui-issues branch November 29, 2023 04:34
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.

2 participants