ui5-list-busy-row should not display when list busy is false

[SherrySherry123]

Hi colleague,
I am from Successfactors team, please help to fix this issue.

Describe the bug
Issue: ui5-list-busy-row always display no matter if list is busy or not. And ui5-list-busy-row is empty with 2rem height.
Expect: ui5-list-busy-row should not display

To reproduce
Steps to reproduce the behavior:

1. Create List with busy and infiniteScroll

Expected behavior
When busy is true, display ui5-list-busy-row;
When busy is false, ui5-list-busy-row should set display as none;

Screenshots

[NHristov-sap]

Hi @SherrySherry123 ,

Can you please give a isolated sample that demonstrates the issue in CodeSandBox?

Best Regards,
Nikolay Hristov
GitHub Dispatcher

[SherrySherry123]

Hi @NHristov-sap,
Here is the sample. Please let me know if you have further question.
https://codesandbox.io/s/ui5-webcomponents-forked-m0207?file=/index.html

Regards,
Sherry

[SherrySherry123]

Hi @Martotko and @NHristov-sap ,
Is there anyone looking at this issue?

Regards,
Sherry

[nnaydenow]

Hi @SherrySherry123,

The busy-row is added when infinity-scroll attribute or busy attribute are set otherwise it is not visible. You could remove the attribute if you are not using this behavior or when the last item is loaded. Seems to be not bug.

Regards,
Nayden

[SherrySherry123]

Hi @nnaydenow ,

Actually we need the infinity-scroll attribute, we need to load more items when the list is long;
As the list items is returned from backend, the list items count maybe 100, or may 1;

So there are two cases for me:

1. When the list is long, when scroll down to the busy-row area, it will show the busy indicator and load more items;

• 

2. When the list is short, as there is no more items and no scroll, we expect the busy-row not display; As it breaks the ui layout in this case, as busy-row display nothing, but occupy an empty space with width and height;

• 
• 

You can see the heigh marked with purple is different for case1 and case2, because the case2 display busy-row;
Please help to fix that, as our goal uxr project depends on it, and we need to deliver user friendly ui;