Skip to content

[DynamicPageTitle]: No Gap Between Buttons in Custom Actions Component in DynamicPageTitle #5873

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

Closed
1 task done
bomasap opened this issue May 30, 2024 · 1 comment · Fixed by #5881
Closed
1 task done

Comments

@bomasap
Copy link

bomasap commented May 30, 2024

Describe the bug

There is an issue with the DynamicPageTitle component's actions property when using a custom actions component. The custom actions component, which includes two buttons, does not display a gap between the buttons when placed in the actions property. However, placing the two buttons directly in the actions property does result in the correct gap between them.

According to another related bug (#5863), this issue is mentioned in the context of a slot. However, the custom actions component does not have a slot in its props.

Isolated Example

https://stackblitz.com/edit/github-zwueh7-t5uuzk?file=src%2FApp.tsx,src%2FCustomActions.tsx

Reproduction steps

1.Place the custom actions component in the actions property of the DynamicPageTitle component. Observe the lack of gap between the two buttons. https://stackblitz.com/edit/github-zwueh7-t5uuzk?file=src%2FApp.tsx,src%2FCustomActions.tsx
2. Place the two buttons directly in the actions property of the DynamicPageTitle component. Observe the presence of a gap between the buttons. https://stackblitz.com/edit/github-zwueh7-weexue?file=src%2FApp.tsx

Expected Behaviour

When the custom actions component is placed in the actions property of the DynamicPageTitle, there should be a gap between the buttons, similar to when the buttons are placed directly in the actions property.

Screenshots or Videos

Screenshot 2024-05-30 at 3 08 10 PM Screenshot 2024-05-30 at 3 08 05 PM

UI5 Web Components for React Version

1.28.0

UI5 Web Components Version

1.24.0

Browser

Chrome, Edge, Firefox, Safari

Operating System

No response

Additional Context

No response

Relevant log output

No response

Organization

No response

Declaration

  • I’m not disclosing any internal or sensitive information.
@Lukas742
Copy link
Contributor

Lukas742 commented Jun 3, 2024

Hi @bomasap

custom components that render multiple elements are currently not supported as actions of the DynamicPageTitle. You can find out more about this here.

The linked PR will outline this in our docs. For updates please subscribe to the corresponding feature request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 🆕 New
Development

Successfully merging a pull request may close this issue.

2 participants