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

feat(Framework): update to ui5-webcomponents-rc.6 #392

Merged
merged 41 commits into from
Apr 2, 2020

Conversation

MarcusNotheis
Copy link
Contributor

@MarcusNotheis MarcusNotheis commented Mar 27, 2020

Features

Please look at the UI5 Web Components Changelog.

Breaking Changes

BREAKING CHANGE: Replace Avatar Component with Avatar UI5 Web Component
BREAKING CHANGE: Replace Carousel Component with Carousel UI5 Web Component
BREAKING CHANGE: Replace SegmentedButton Component with SegmentedButton UI5 Web Component
BREAKING CHANGE: Deleted SegmentedButtonItem, replaced by ToggleButton
BREAKING CHANGE: Remove MessageToast Component, replaced by Toast UI5 Web Component
BREAKING CHANGE: ThemeProvider: Remove prop withToastContainer
BREAKING CHANGE: AvatarSize: Remove value Custom
BREAKING CHANGE: ButtonDesign: RenameAccept to Positive
BREAKING CHANGE: ButtonDesign: RenameReject to Negative
BREAKING CHANGE: InputType: Rename Url to URL
BREAKING CHANGE: CarouselArrowsPlacement: Rename PageIndicator to Navigation
BREAKING CHANGE:BusyIndicator : change default size of busy dots to Medium
BREAKING CHANGE:Icon : src property is removed in favor of name
BREAKING CHANGE:MessageStrip : replace icon property with icon slot to allow setting arbitrary content by the user
BREAKING CHANGE:MultiComboBox : Does not longer accept StandardListItem as child, use MultiComboBoxItem instead
BREAKING CHANGE:ShellBar : profile property is removed in favour of profile slot
BREAKING CHANGE:TabContainer : onItemSelect event is renamed to onTabSelect and the item event parameter is renamed to tab.
BREAKING CHANGE:Card : the property subtitle has been renamed to subheading
BREAKING CHANGE:TextArea : the property maxLength has been renamed to maxlength
BREAKING CHANGE:ThemeProvider : new prop theme is required for correct theming in addition to importing import '@ui5/webcomponents-react/lib/ThemingSupport'
BREAKING CHANGE: configuring of compact size is removed, use the ui5-content-density-compact CSS class to apply compact size.

Closes #349

@github-actions
Copy link

Coverage Status

Coverage decreased (-1.4%) to 68.636% when pulling d4c64d7 on chore/update-webcomponents-rc-6 into 69e648a on master.

@MarcusNotheis MarcusNotheis marked this pull request as ready for review March 29, 2020 08:05
BREAKING CHANGE: Remove AvatarSize `Custom`
BREAKING CHANGE: Rename ButtonDesign `Accept` to `Positive`
BREAKING CHANGE: Rename ButtonDesign `Reject` to `Negative`
BREAKING CHANGE: Rename InputType `Url` to `URL`
Copy link
Contributor

@vbersch vbersch left a comment

Choose a reason for hiding this comment

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

  • BusyIndicator story: default value for active should be true
  • Card story: Card looks different than in ui5-webcomponents demo
  • list of BREAKING CHANGES is missing
  • whats happening with the ComboBox story?
  • Popover story: modal and noArrow should be false by default; same for ResponsivePopover
  • Should we really keep two different toast components?

BREAKING CHANGE: Remove MessageToast Component, replaced by `Toast` UI5 Web Component
BREAKING CHANGE: Remove prop `withToastContainer` from ThemeProvider
@Lukas742
Copy link
Contributor

Lukas742 commented Mar 30, 2020

  • Carousel doesn't show content for index > 0 if dir="rtl"

  • SegmentedButton dir="rtl" breaks layout.

  • Card dir="rtl" breaks layout.

  • name prop needs @ui5/webcomponents/dist/features/InputElementsFormSupport.js import

  • DatePicker and Calendar: Props maxDate, minDate, primaryCalendarType, selectedDates are throwing errors when set.

  • MessageStrip: Icon stays on left position if dir="rtl" is set

@MarcusNotheis
Copy link
Contributor Author

  • whats happening with the ComboBox story?
    What exactly are you referring to @vbersch?

@vbersch
Copy link
Contributor

vbersch commented Mar 30, 2020

  • whats happening with the ComboBox story?
    What exactly are you referring to @vbersch?

I was referring to the storybook shortcuts 't' and 's' that we talked about. Is there anything we can do about this at the moment? Thats a storybook bug, isn´t it?

@MarcusNotheis MarcusNotheis changed the title Chore/update webcomponents rc 6 feat(Framework): update to ui5-webcomponents-rc.6 Apr 1, 2020
@netlify
Copy link

netlify bot commented Apr 1, 2020

Deploy preview for ui5-webcomponents-react ready!

Built with commit 64b9951

https://deploy-preview-392--ui5-webcomponents-react.netlify.com

@MarcusNotheis MarcusNotheis merged commit 84a3f90 into master Apr 2, 2020
@MarcusNotheis MarcusNotheis deleted the chore/update-webcomponents-rc-6 branch April 2, 2020 11:26
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.

Update UI5 Web Components to rc.6
3 participants