-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
sap.ui.layout.form.SimpleForm Aggregation Clone Issue #4015
Comments
Thanks for reporting this issue. In UI5 hidden aggregations are not part of clones. The SimpleForm uses the form as a hidden aggregation and this seems like the main issue here. Internal incident DINC0107232 is created for this github issue. |
Hi, this issue is already solved with SAPUI5 version 1.121 |
Fixed with a8a8199 |
@flovogt Thank you ;) Why is this fix not in the long-term maintenance version |
Hello @edemuck4timbuktu, |
@edemuck4timbuktu : Could you please tell us what SAP product has an problem? We have strict downport guidelines to keep running installations stable. Only if there is a real issue with a real application a downpord to older SAPUI5 versions is done. |
@UserAbcd1234 It is ok, thanks. I will be wait to the next Long-term Maintenance version. |
Hello @edemuck4timbuktu, Thank you for your response! Given all the communication, I think it is appropriate to mark the ticket as resolved. Best regards, |
OpenUI5 version:
1.120.11
Browser/version (+device/version):
Chrome Version 122.0.6261.131 (Offizieller Build) (64-Bit)
URL (minimal example if possible):
https://plnkr.co/plunk/uyRhWUvienyHvQ1p
Steps to reproduce the problem:
What is the expected result?
The
sap.m.OverflowToolbar
of eachsap.ui.layout.form.SimpleForm
should be displayed. Only the lastsap.ui.layout.form.SimpleForm
displayed the toolbar.What happens instead?
The
sap.m.OverflowToolbar
of thesap.ui.layout.form.SimpleForm
, which is the content of asap.f. Card
, isn't displayed.Any other information? (attach screenshot if possible)
First analysis:
The cloning of the
sap.ui.layout.form.SimpleForm
triggers by rendering of the aggregationpages
of thesap.m.Carousel
isn't working properly.After cloning the
SimpleForm
, the aggregationtoolbar
isn't part of the newSimpleForm
. See https://github.com/SAP/openui5/blob/1.120.11/src/sap.ui.layout/src/sap/ui/layout/form/SimpleForm.js#L1141The text was updated successfully, but these errors were encountered: