fix(DurationPicker): use correct value for defaultProp maxValue by Lukas742 · Pull Request #529 · UI5/webcomponents-react

Lukas742 · 2020-05-20T11:35:05Z

Thank you for your contribution! 👏

To get it merged faster, kindly review the checklist below:

Pull Request Checklist

Reviewed the Contributing Guidelines
- Especially the How to Contribute section
Correct commit message style

github-actions · 2020-05-20T11:39:15Z

Coverage remained the same at 65.899% when pulling 2d2a410 on fix/duration-picker-default-max-value into 1154b35 on master.

github-actions · 2020-05-20T11:39:58Z

Size Change: -3 B (0%)

Total Size: 143 kB

Filename	Size	Change
`packages/main/lib/DurationPicker.js`	395 B	-3 B (0%)

ℹ️ View Unchanged

Filename	Size	Change
`packages/base/lib/createComponentStyles.js`	206 B	0 B
`packages/base/lib/CssSizeVariables.js`	820 B	0 B
`packages/base/lib/Device.js`	6.47 kB	0 B
`packages/base/lib/hooks.js`	1.19 kB	0 B
`packages/base/lib/Logger.js`	1.2 kB	0 B
`packages/base/lib/spacing.js`	853 B	0 B
`packages/base/lib/StyleClassHelper.js`	570 B	0 B
`packages/base/lib/ThemingParameters.js`	3.4 kB	0 B
`packages/base/lib/useConsolidatedRef.js`	271 B	0 B
`packages/base/lib/usePassThroughHtmlProps.js`	452 B	0 B
`packages/base/lib/useViewportRange.js`	460 B	0 B
`packages/base/lib/Utils.js`	774 B	0 B
`packages/charts/lib/BarChart.js`	3.59 kB	0 B
`packages/charts/lib/BarChartPlaceholder.js`	456 B	0 B
`packages/charts/lib/ColumnChart.js`	3.76 kB	0 B
`packages/charts/lib/ColumnChartPlaceholder.js`	514 B	0 B
`packages/charts/lib/components/ChartContainer.js`	877 B	0 B
`packages/charts/lib/components/ChartDataLabel.js`	604 B	0 B
`packages/charts/lib/components/ComposedChartPlaceholder.js`	878 B	0 B
`packages/charts/lib/components/XAxisTicks.js`	596 B	0 B
`packages/charts/lib/components/YAxisTicks.js`	513 B	0 B
`packages/charts/lib/ComposedChart.js`	4.56 kB	0 B
`packages/charts/lib/DonutChart.js`	1.89 kB	0 B
`packages/charts/lib/LineChart.js`	3.71 kB	0 B
`packages/charts/lib/LineChartPlaceholder.js`	529 B	0 B
`packages/charts/lib/MicroBarChart.js`	1.9 kB	0 B
`packages/charts/lib/PieChart.js`	1.81 kB	0 B
`packages/charts/lib/PieChartPlaceholder.js`	445 B	0 B
`packages/charts/lib/RadarChart.js`	1.89 kB	0 B
`packages/charts/lib/RadialChart.js`	1.06 kB	0 B
`packages/charts/lib/useLegendItemClick.js`	391 B	0 B
`packages/charts/lib/Utils.js`	534 B	0 B
`packages/main/lib/ActionSheet.js`	1.25 kB	0 B
`packages/main/lib/AnalyticalCard.js`	774 B	0 B
`packages/main/lib/AnalyticalCardHeader.js`	2.67 kB	0 B
`packages/main/lib/AnalyticalTable.js`	15.8 kB	0 B
`packages/main/lib/AnalyticalTableScrollMode.js`	177 B	0 B
`packages/main/lib/Avatar.js`	438 B	0 B
`packages/main/lib/AvatarBackgroundColor.js`	219 B	0 B
`packages/main/lib/AvatarFitType.js`	151 B	0 B
`packages/main/lib/AvatarShape.js`	150 B	0 B
`packages/main/lib/AvatarSize.js`	155 B	0 B
`packages/main/lib/Badge.js`	320 B	0 B
`packages/main/lib/Bar.js`	1.3 kB	0 B
`packages/main/lib/BarDesign.js`	177 B	0 B
`packages/main/lib/Breadcrumbs.js`	740 B	0 B
`packages/main/lib/BreadcrumbsSeparatorStyle.js`	198 B	0 B
`packages/main/lib/BusyIndicator.js`	357 B	0 B
`packages/main/lib/BusyIndicatorSize.js`	167 B	0 B
`packages/main/lib/Button.js`	370 B	0 B
`packages/main/lib/ButtonDesign.js`	190 B	0 B
`packages/main/lib/Calendar.js`	348 B	0 B
`packages/main/lib/CalendarType.js`	212 B	0 B
`packages/main/lib/Card.js`	342 B	0 B
`packages/main/lib/Carousel.js`	418 B	0 B
`packages/main/lib/CarouselArrowsPlacement.js`	166 B	0 B
`packages/main/lib/CheckBox.js`	388 B	0 B
`packages/main/lib/ComboBox.js`	431 B	0 B
`packages/main/lib/ComboBoxItem.js`	311 B	0 B
`packages/main/lib/ContentDensity.js`	151 B	0 B
`packages/main/lib/CurrentViewportRangeContext.js`	133 B	0 B
`packages/main/lib/CustomListItem.js`	355 B	0 B
`packages/main/lib/DatePicker.js`	459 B	0 B
`packages/main/lib/DateTimePicker.js`	467 B	0 B
`packages/main/lib/DeviationIndicator.js`	160 B	0 B
`packages/main/lib/Dialog.js`	364 B	0 B
`packages/main/lib/FileUploader.js`	401 B	0 B
`packages/main/lib/FilterBar.js`	1.36 kB	0 B
`packages/main/lib/FilterItem.js`	1.54 kB	0 B
`packages/main/lib/FilterType.js`	170 B	0 B
`packages/main/lib/FlexBox.js`	991 B	0 B
`packages/main/lib/FlexBoxAlignItems.js`	189 B	0 B
`packages/main/lib/FlexBoxDirection.js`	169 B	0 B
`packages/main/lib/FlexBoxJustifyContent.js`	194 B	0 B
`packages/main/lib/FlexBoxWrap.js`	154 B	0 B
`packages/main/lib/Form.js`	1.16 kB	0 B
`packages/main/lib/FormGroup.js`	559 B	0 B
`packages/main/lib/FormItem.js`	1.08 kB	0 B
`packages/main/lib/GlobalStyleClasses.js`	151 B	0 B
`packages/main/lib/Grid.js`	1.6 kB	0 B
`packages/main/lib/GridPosition.js`	157 B	0 B
`packages/main/lib/GroupHeaderListItem.js`	333 B	0 B
`packages/main/lib/Icon.js`	328 B	0 B
`packages/main/lib/Input.js`	465 B	0 B
`packages/main/lib/InputType.js`	184 B	0 B
`packages/main/lib/Label.js`	341 B	0 B
`packages/main/lib/Link.js`	359 B	0 B
`packages/main/lib/LinkDesign.js`	191 B	0 B
`packages/main/lib/List.js`	439 B	0 B
`packages/main/lib/ListItemTypes.js`	160 B	0 B
`packages/main/lib/ListMode.js`	191 B	0 B
`packages/main/lib/ListSeparators.js`	186 B	0 B
`packages/main/lib/Loader.js`	1.3 kB	0 B
`packages/main/lib/LoaderType.js`	152 B	0 B
`packages/main/lib/MessageBox.js`	2.24 kB	0 B
`packages/main/lib/MessageBoxActions.js`	231 B	0 B
`packages/main/lib/MessageBoxTypes.js`	232 B	0 B
`packages/main/lib/MessageStrip.js`	371 B	0 B
`packages/main/lib/MessageStripType.js`	181 B	0 B
`packages/main/lib/MultiComboBox.js`	438 B	0 B
`packages/main/lib/MultiComboBoxItem.js`	330 B	0 B
`packages/main/lib/Notification.js`	3.41 kB	0 B
`packages/main/lib/NotificationGroup.js`	339 B	0 B
`packages/main/lib/ObjectPage.js`	9.22 kB	0 B
`packages/main/lib/ObjectPageMode.js`	159 B	0 B
`packages/main/lib/ObjectPageSection.js`	1.27 kB	0 B
`packages/main/lib/ObjectPageSubSection.js`	1.11 kB	0 B
`packages/main/lib/ObjectStatus.js`	1.22 kB	0 B
`packages/main/lib/Option.js`	324 B	0 B
`packages/main/lib/Page.js`	2.05 kB	0 B
`packages/main/lib/PageBackgroundDesign.js`	183 B	0 B
`packages/main/lib/Panel.js`	399 B	0 B
`packages/main/lib/PanelAccessibleRoles.js`	201 B	0 B
`packages/main/lib/PlacementType.js`	175 B	0 B
`packages/main/lib/Popover.js`	487 B	0 B
`packages/main/lib/PopoverHorizontalAlign.js`	181 B	0 B
`packages/main/lib/PopoverPlacementType.js`	172 B	0 B
`packages/main/lib/PopoverVerticalAlign.js`	204 B	0 B
`packages/main/lib/Priority.js`	163 B	0 B
`packages/main/lib/ProductSwitch.js`	308 B	0 B
`packages/main/lib/ProductSwitchItem.js`	359 B	0 B
`packages/main/lib/ProgressIndicator.js`	1.34 kB	0 B
`packages/main/lib/RadioButton.js`	397 B	0 B
`packages/main/lib/ResponsivePopover.js`	502 B	0 B
`packages/main/lib/SegmentedButton.js`	321 B	0 B
`packages/main/lib/Select.js`	352 B	0 B
`packages/main/lib/SemanticColor.js`	181 B	0 B
`packages/main/lib/ShellBar.js`	447 B	0 B
`packages/main/lib/ShellBarItem.js`	335 B	0 B
`packages/main/lib/SideNavigation.js`	1.44 kB	0 B
`packages/main/lib/SideNavigationListItem.js`	2.11 kB	0 B
`packages/main/lib/SideNavigationOpenState.js`	174 B	0 B
`packages/main/lib/Size.js`	151 B	0 B
`packages/main/lib/Spinner.js`	1.2 kB	0 B
`packages/main/lib/StandardListItem.js`	417 B	0 B
`packages/main/lib/SuggestionItem.js`	386 B	0 B
`packages/main/lib/Switch.js`	351 B	0 B
`packages/main/lib/Tab.js`	372 B	0 B
`packages/main/lib/TabContainer.js`	412 B	0 B
`packages/main/lib/TabContainerTabsPlacement.js`	161 B	0 B
`packages/main/lib/TabLayout.js`	150 B	0 B
`packages/main/lib/Table.js`	365 B	0 B
`packages/main/lib/TableCell.js`	303 B	0 B
`packages/main/lib/TableColumn.js`	352 B	0 B
`packages/main/lib/TablePlaceholder.js`	771 B	0 B
`packages/main/lib/TableRow.js`	302 B	0 B
`packages/main/lib/TableScaleWidthMode.js`	169 B	0 B
`packages/main/lib/TableSelectionBehavior.js`	178 B	0 B
`packages/main/lib/TableSelectionMode.js`	182 B	0 B
`packages/main/lib/TabSeparator.js`	306 B	0 B
`packages/main/lib/Text.js`	824 B	0 B
`packages/main/lib/TextAlign.js`	183 B	0 B
`packages/main/lib/TextArea.js`	469 B	0 B
`packages/main/lib/ThemeProvider.js`	1.09 kB	0 B
`packages/main/lib/Themes.js`	178 B	0 B
`packages/main/lib/Timeline.js`	298 B	0 B
`packages/main/lib/TimelineItem.js`	351 B	0 B
`packages/main/lib/TimePicker.js`	411 B	0 B
`packages/main/lib/Title.js`	337 B	0 B
`packages/main/lib/TitleLevel.js`	163 B	0 B
`packages/main/lib/Toast.js`	349 B	0 B
`packages/main/lib/ToastPlacement.js`	208 B	0 B
`packages/main/lib/ToggleButton.js`	391 B	0 B
`packages/main/lib/Token.js`	233 B	0 B
`packages/main/lib/Tokenizer.js`	248 B	0 B
`packages/main/lib/UploadCollection.js`	398 B	0 B
`packages/main/lib/UploadCollectionItem.js`	449 B	0 B
`packages/main/lib/UploadState.js`	170 B	0 B
`packages/main/lib/ValueState.js`	182 B	0 B
`packages/main/lib/VariantManagement.js`	1.92 kB	0 B
`packages/main/lib/VerticalAlign.js`	174 B	0 B
`packages/main/lib/withWebComponent.js`	1.55 kB	0 B

_{compressed-size-action}

fix(DurationPicker): correct value for defaultProp maxValue

2d2a410

Lukas742 requested a review from MarcusNotheis May 20, 2020 11:35

remove maxValue from defaultProps

7f2c536

MarcusNotheis approved these changes May 20, 2020

View reviewed changes

Lukas742 merged commit 888d069 into master May 20, 2020

Lukas742 deleted the fix/duration-picker-default-max-value branch May 20, 2020 11:53

MarcusNotheis pushed a commit that referenced this pull request May 20, 2020

fix(DurationPicker): use correct value for defaultProp maxValue (#529)

c3bb684

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix(DurationPicker): use correct value for defaultProp maxValue#529

fix(DurationPicker): use correct value for defaultProp maxValue#529
Lukas742 merged 2 commits into
masterfrom
fix/duration-picker-default-max-value

Lukas742 commented May 20, 2020 •

edited

Loading

Uh oh!

github-actions Bot commented May 20, 2020

Uh oh!

github-actions Bot commented May 20, 2020 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

Lukas742 commented May 20, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Pull Request Checklist

Uh oh!

github-actions Bot commented May 20, 2020

Uh oh!

github-actions Bot commented May 20, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Lukas742 commented May 20, 2020 •

edited

Loading

github-actions Bot commented May 20, 2020 •

edited

Loading