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

rForbiddenTags Prevents ColumnListItem to Switch Its Busy State #2041

Closed
boghyon opened this issue May 24, 2018 · 3 comments
Closed

rForbiddenTags Prevents ColumnListItem to Switch Its Busy State #2041

boghyon opened this issue May 24, 2018 · 3 comments
Assignees
Labels

Comments

@boghyon
Copy link
Contributor

boghyon commented May 24, 2018

URL (minimal example if possible):

https://openui5.hana.ondemand.com/#/sample/sap.m.sample.Table/preview

Steps to reproduce the problem:

  1. Open the above example page
  2. Set the busy state of one of the ColumnListItems to true

What is the expected result?

The item becomes busy and displays a BusyIndicator in it.

What happens instead?

Since ColumnListItem is rendered with the HTML tag <tr>, which is one of the forbidden tags, enabling busy state is skipped.

rForbiddenTags = /^(?:area|base|br|col|embed|hr|img|input|keygen|link|menuitem|meta|param|source|track|wbr|tr)$/i;

Any other information? (attach screenshot if possible)

ui5 tr busy indicator warning

@myordanov myordanov self-assigned this Jun 4, 2018
@myordanov
Copy link
Contributor

Hello @boghyon,
Thank you for sharing this finding. I've created an internal incident 1880272398. The status of the issue will be updated here in GitHub.
Regards, Mihail.

@aborjinik
Copy link
Contributor

@boghyon this is a known limitation as you already see the console message. we do not support busy for the ColumnListItem

@vxmn
Copy link

vxmn commented Sep 9, 2020

I would like to know the reasoning, why it is not possible to set the busy indicator for this control. From my point of view it makes sense to mark a list item as busy.

Also, even though there is a console message to indicate that the BusyIndicator is not supported, the official documentation does not mention this limitation.

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

No branches or pull requests

5 participants