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: extending sidepanel example to include rounded corners + new icon #270

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ethanwinters
Copy link
Member

Also adding a fullscreen example.

Also fixed a bug in the example where we were leaving the parent element still on the screen blocking clicks.

Cannot be merged until 8.2 goes out with rounded corner support.

Also adding a fullscreen example.

Also fixed a bug in the example where we were leaving the parent element still on the screen blocking clicks.

Can not be merged until 8.2 goes out with rounded corner support.
@@ -81,6 +85,8 @@
// Move the main window to the off-screen position and then un-hide it.
instance.elements.getMainWindow().addClassName('StartOpenAnimation');
instance.elements.getMainWindow().removeClassName('HideWebChat');
// Make the custom element visible.
customElement.classList.remove('WebChatContainer--hidden');
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think this should be here. One of the points of this tutorial is to demonstrate how to animate the opening and closing of web chat and still use our built-in launcher. If you hide the custom element, you can't do that. And in fact, because of this, this example doesn't actually work. If you open this html file as-is, the custom element is hidden and the launcher is invisible.

Copy link
Contributor

Choose a reason for hiding this comment

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

If I remove the changes to the hide the custom element, the entrance animation doesn't work. Web chat just instantly appears. The exit animation does however work.

// Have the "close" button be an icon to show that the chat will close to the right.
closeButtonIconType: 'side-panel-right'
},
carbonTheme: {
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be themeConfig, not carbonConfig.

element: customElement,
// Move to fullscreen layout option.
layout: {
// Removes dropshadows and borders.
Copy link
Contributor

Choose a reason for hiding this comment

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

Typo: dropshadows -> drop shadows

@@ -47,6 +48,8 @@ function viewChangeHandler(event, instance, stylesInitializedRef) {
// Move the main window to the off-screen position and then un-hide it.
instance.elements.getMainWindow().addClassName('StartOpenAnimation');
instance.elements.getMainWindow().removeClassName('HideWebChat');
// Make the custom element visible.
customElement.classList.remove('WebChatContainer--hidden');
Copy link
Contributor

Choose a reason for hiding this comment

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

In this case, the launcher is visible when you first launch the app and you can use it to open web chat but when you close the web chat, the launcher does not become visible again and you can't open web chat back up.

@@ -6,6 +6,9 @@ const config = {
region: 'us-south',
serviceInstanceID: '9a3613d2-3ce6-4928-8eb6-4d659d87ae68',
subscriptionID: null,
// You would likely use our own launcher in this scenario.
Copy link
Contributor

Choose a reason for hiding this comment

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

If you want square corners like the JS example, you should add the corners option.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants