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

Unexpected 2px "padding" in Dialog #2694

Closed
pavelkornev opened this issue Jan 16, 2021 · 1 comment · Fixed by #2770
Closed

Unexpected 2px "padding" in Dialog #2694

pavelkornev opened this issue Jan 16, 2021 · 1 comment · Fixed by #2770
Assignees
Labels
bug This issue is a bug in the code Medium Prio SAP Graph TOPIC RD

Comments

@pavelkornev
Copy link
Member

pavelkornev commented Jan 16, 2021

Describe the bug
When there is no headerText and there is nothing inside header slot, then 2px padding remains at the top of the dialog.

To reproduce

  1. Create dialog without headerText and without header slot
  2. Check the resulting rendering (see sandbox link below)

Isolated example
https://codesandbox.io/s/ui5-webcomponents-forked-gclih?file=/index.html

Expected behavior
ui5-popup-header-root is set to display: none when there is no headerText and there is nothing inside header slot

<header
class="ui5-popup-header-root"
id="ui5-popup-header"
@mousedown="{{_onDragMouseDown}}">
{{#if header.length }}
<slot name="header"></slot>
{{else}}
<h2 class="ui5-popup-header-text">{{headerText}}</h2>
{{/if}}
</header>

Screenshots
Screen Shot 2021-01-16 at 21 04 52

Affected components

  • Dialog
  • Popover

Organization: SAP Graph

Priority: Medium

@nnaydenow
Copy link
Contributor

Hello @SAP/ui5-webcomponents-topic-rd,

As @pavelkornev mention header ui5-popup-header-root is still visible even it is empty and this leads to visual misalignments.

The issue is reproducible in:
https://codesandbox.io/s/ui5-webcomponents-forked-gclih?file=/index.html

Regards,
Nayden

@nnaydenow nnaydenow added bug This issue is a bug in the code SAP Graph TOPIC RD labels Jan 17, 2021
@nnaydenow nnaydenow added this to Issues in Maintenance - Topic RD via automation Jan 17, 2021
@georgimkv georgimkv self-assigned this Feb 2, 2021
@ilhan007 ilhan007 moved this from New Issues to In Progress in Maintenance - Topic RD Feb 3, 2021
Maintenance - Topic RD automation moved this from In Progress to Completed Feb 3, 2021
NHristov-sap pushed a commit to NHristov-sap/ui5-webcomponents that referenced this issue Feb 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug in the code Medium Prio SAP Graph TOPIC RD
Projects
Status: Completed
Development

Successfully merging a pull request may close this issue.

3 participants