SAP / openui5

OpenUI5 lets you build enterprise-ready web applications, responsive to all devices, running on almost any browser of your choice.
http://openui5.org
Apache License 2.0
2.9k stars 1.23k forks source link

Dialog Cancel Buttons have Border around them in 1.119 and 1.120 #4046

Closed ac1058 closed 1 week ago

ac1058 commented 2 months ago

OpenUI5 version: 1.119, 1.120

Browser/version (+device/version): Chrome

Any other tested browsers/devices(OK/FAIL): N/A

URL (minimal example if possible):

User/password (if required and possible - do not post any confidential information here):

Steps to reproduce the problem:

  1. open the sap.m.Dialog sample in UI5 sample docs
    • click Settings Cog in top right and select 'Belize' and 'Compact'
  2. Observe that in 1.119.x and 1.120.x the "Cancel" button has a border around it that did not exist in previous versions and versions >= 1.121.x

What is the expected result?

What happens instead?

Any other information? (attach screenshot if possible)

DonkeyCo commented 1 month ago

Thanks for reporting the issue. I've created an internal incident DINC0148190 for your reported issue. The colleagues will update you once they have news

stephania87 commented 1 week ago

sap_belize is deprecated since SAPUI5 1.120: https://openui5nightly.hana.ondemand.com/topic/a87ca843bcee469f82a9072927a7dcdb And the rest of the listed cloud only versions are out of maintenance now https://ui5.sap.com/versionoverview.html Please upgrade and switch to sap_horizon.

Just fyi, the changes are related to new themes development where sap.m.button[type=Default] is not changed when used in specific toolbars: https://github.com/SAP/openui5/commit/ebe17c6c3c054a4922e02a4dfc896d6ef4592c49

stephania87 commented 1 week ago

PS: when you create toolbars with buttons, consider setting Toolbar.design and Button.type based on https://experience.sap.com/fiori-design-web/toolbar-overview/#header-and-footer-toolbars1 As for demos, there are already logged internal tasks for reviewing the settings in the DemoKit with consideration of Fiori Guidelines