Skip to content

Conversation

@unazko
Copy link
Contributor

@unazko unazko commented Apr 4, 2023

New properties "busy" and "busyDelay" have been introduced.
Through those properties the application developers
will be able to configure a busy indicator display for a particular ui5-menu
DOM representation. That way there will be a busy indicator while
the corresponding menu item texts are getting fetched from a database.

Fixes: #6131

New property "busyIndicator" has been introduced.
Through this property the application developers
will be able to configure a busy indicator display
for a particular ui5-menu DOM representation.
That way there will be a busy indicator while
the corresponding menu item texts are getting fetched
from a database.

Fixes: UI5#6131
New property "busyIndicator" has been introduced.
Through this property the application developers
will be able to configure a busy indicator display for a particular ui5-menu
DOM representation. That way there will be a busy indicator while
the corresponding menu item texts are getting fetched from a database.

Fixes: UI5#6131
Copy link
Contributor

@ilhan007 ilhan007 left a comment

Choose a reason for hiding this comment

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

Clicking on "Request changes" just to be sure the core team have some time to do API review of this feature, @tsanislavgatev @NHristov-sap please continue with the review normally

unazko added 2 commits April 4, 2023 19:40
New property "busyIndicator" has been introduced.
Through this property the application developers
will be able to configure a busy indicator display for a particular ui5-menu
DOM representation. That way there will be a busy indicator while
the corresponding menu item texts are getting fetched from a database.

Fixes: UI5#6131
New property "busyIndicator" has been introduced.
Through this property the application developers
will be able to configure a busy indicator display for a particular ui5-menu
DOM representation. That way there will be a busy indicator while
the corresponding menu item texts are getting fetched from a database.

Fixes: UI5#6131
@unazko unazko requested a review from DavidMitreski5 April 4, 2023 16:51
unazko added 4 commits April 4, 2023 20:28
New property "busyIndicator" has been introduced.
Through this property the application developers
will be able to configure a busy indicator display for a particular ui5-menu
DOM representation. That way there will be a busy indicator while
the corresponding menu item texts are getting fetched from a database.

Fixes: UI5#6131
New property "busyIndicator" has been introduced.
Through this property the application developers
will be able to configure a busy indicator display for a particular ui5-menu
DOM representation. That way there will be a busy indicator while
the corresponding menu item texts are getting fetched from a database.

Fixes: UI5#6131
New property "busyIndicator" has been introduced.
Through this property the application developers
will be able to configure a busy indicator display for a particular ui5-menu
DOM representation. That way there will be a busy indicator while
the corresponding menu item texts are getting fetched from a database.

Fixes: UI5#6131
New property "busyIndicator" has been introduced.
Through this property the application developers
will be able to configure a busy indicator display for a particular ui5-menu
DOM representation. That way there will be a busy indicator while
the corresponding menu item texts are getting fetched from a database.

Fixes: UI5#6131
unazko added 2 commits April 5, 2023 11:18
New property "busyIndicator" has been introduced.
Through this property the application developers
will be able to configure a busy indicator display for a particular ui5-menu
DOM representation. That way there will be a busy indicator while
the corresponding menu item texts are getting fetched from a database.

Fixes: UI5#6131
New property "busyIndicator" has been introduced.
Through this property the application developers
will be able to configure a busy indicator display for a particular ui5-menu
DOM representation. That way there will be a busy indicator while
the corresponding menu item texts are getting fetched from a database.

Fixes: UI5#6131
Two new properties "busy" and "busyDelay" have been introduced.
Through those properties the application developers
will be able to configure a busy indicator display for a particular ui5-menu
DOM representation. That way there will be a busy indicator while
the corresponding menu item texts are getting fetched from a database.

Fixes: UI5#6131
New properties "busy" and "busyDelay" have been introduced.
Through those properties the application developers
will be able to configure a busy indicator display for a particular ui5-menu
DOM representation. That way there will be a busy indicator while
the corresponding menu item texts are getting fetched from a database.

Fixes: UI5#6131
unazko added 2 commits April 13, 2023 19:08
New properties "busy" and "busyDelay" have been introduced.
Through those properties the application developers
will be able to configure a busy indicator display for a particular ui5-menu
DOM representation. That way there will be a busy indicator while
the corresponding menu item texts are getting fetched from a database.

Fixes: UI5#6131
New properties "busy" and "busyDelay" have been introduced.
Through those properties the application developers
will be able to configure a busy indicator display for a particular ui5-menu
DOM representation. That way there will be a busy indicator while
the corresponding menu item texts are getting fetched from a database.

Fixes: UI5#6131
unazko added 2 commits April 18, 2023 12:09
New properties "busy" and "busyDelay" have been introduced.
Through those properties the application developers
will be able to configure a busy indicator display for a particular ui5-menu
DOM representation. That way there will be a busy indicator while
the corresponding menu item texts are getting fetched from a database.

Fixes: UI5#6131
New properties "busy" and "busyDelay" have been introduced.
Through those properties the application developers
will be able to configure a busy indicator display for a particular ui5-menu
DOM representation. That way there will be a busy indicator while
the corresponding menu item texts are getting fetched from a database.

Fixes: UI5#6131
@unazko unazko requested a review from DavidMitreski5 April 18, 2023 10:06
unazko added 2 commits April 18, 2023 13:18
New properties "busy" and "busyDelay" have been introduced.
Through those properties the application developers
will be able to configure a busy indicator display for a particular ui5-menu
DOM representation. That way there will be a busy indicator while
the corresponding menu item texts are getting fetched from a database.

Fixes: UI5#6131
New properties "busy" and "busyDelay" have been introduced.
Through those properties the application developers
will be able to configure a busy indicator display for a particular ui5-menu
DOM representation. That way there will be a busy indicator while
the corresponding menu item texts are getting fetched from a database.

Fixes: UI5#6131
unazko added 3 commits April 18, 2023 17:25
New properties "busy" and "busyDelay" have been introduced.
Through those properties the application developers
will be able to configure a busy indicator display for a particular ui5-menu
DOM representation. That way there will be a busy indicator while
the corresponding menu item texts are getting fetched from a database.

Fixes: UI5#6131
New properties "busy" and "busyDelay" have been introduced.
Through those properties the application developers
will be able to configure a busy indicator display for a particular ui5-menu
DOM representation. That way there will be a busy indicator while
the corresponding menu item texts are getting fetched from a database.

Fixes: UI5#6131
New properties "busy" and "busyDelay" have been introduced.
Through those properties the application developers
will be able to configure a busy indicator display for a particular ui5-menu
DOM representation. That way there will be a busy indicator while
the corresponding menu item texts are getting fetched from a database.

Fixes: UI5#6131
@vladitasev vladitasev dismissed stale reviews from nnaydenow and ilhan007 April 20, 2023 12:00

done

@unazko unazko merged commit deb6685 into UI5:main Apr 20, 2023
NHristov-sap pushed a commit that referenced this pull request May 4, 2023
* feat(ui5-menu): provide busy indicator display

New property "busyIndicator" has been introduced.
Through this property the application developers
will be able to configure a busy indicator display
for a particular ui5-menu DOM representation.
That way there will be a busy indicator while
the corresponding menu item texts are getting fetched
from a database.

Fixes: #6131
PetyaMarkovaBogdanova pushed a commit that referenced this pull request Jun 20, 2023
* feat(ui5-menu): provide busy indicator display

New property "busyIndicator" has been introduced.
Through this property the application developers
will be able to configure a busy indicator display
for a particular ui5-menu DOM representation.
That way there will be a busy indicator while
the corresponding menu item texts are getting fetched
from a database.

Fixes: #6131
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dynamically show sub-menu with busy indicator

5 participants