-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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: adjust progress bar wrapper height on web #3997
Conversation
Hey @lukewalczak, thank you for your pull request 🤗. The documentation from this branch can be viewed here. |
The mobile version of example app from this branch is ready! You can see it here. |
@@ -59,3 +59,13 @@ it('renders indeterminate progress bar', async () => { | |||
|
|||
expect(tree.toJSON()).toMatchSnapshot(); | |||
}); | |||
|
|||
it('renders progress bar with full height on web', () => { | |||
Platform.OS = 'web'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wouldn't that affect every test from now on in this file?
@@ -94,11 +94,23 @@ const ProgressBarExample = () => { | |||
<View style={styles.row}> | |||
<Text variant="bodyMedium">ProgressBar with animated value</Text> | |||
<AnimatedProgressBar | |||
visible={visible} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
8208f26
to
64488a8
Compare
Fixes: #3984
Summary
Correct view wrapper styles for the web to properly display
ProgressBar
with the percentage height.Test plan
Added new example and unit test case.