umbraco / Umbraco-CMS

Umbraco is a free and open source .NET content management system helping you deliver delightful digital experiences.
https://umbraco.com
Other
4.49k stars 2.69k forks source link

Save button doesn't work in Content Type Editor (when using non-live MB mode) #9286

Closed skttl closed 3 years ago

skttl commented 4 years ago

When using a non-live ModelsBuilder mode, the save button in the content type editor is changed to a split button, where you can save and generate models. image

Clicking Save here doesn't do anything. If I choose save and generate models, it saves and generates models as expected. After this, the save button is reset to the original state (without the split), and then the save button works. again.

FpYcMN2w0j

Umbraco version

I am seeing this issue on Umbraco version: 8.9.0

skttl commented 4 years ago

Seems like the split button is missing a click-handler image

This is how the regular button looks image

nathanwoulfe commented 4 years ago

@skttl that makes sense - buttons work a lot better when they have something to do...

Marking it up-for-grabs 😄

bjarnef commented 4 years ago

I am seing this issue as well in Umbraco v8.9.0 after switching ModelsBuilder mode from default PureLive mode to AppData mode. It seems the save button is showing the spinner sometimes after the application has been recycled (e.g. via ModelsBuilder "Generate models" button), but not on subsequent clicks.

DanDiplo commented 3 years ago

Yep, I'm getting this with AppData mode. Clicking button does absolutely nothing.

Surely this is a critical issue and not an "up-for-grabs" issue, though, as it completely breaks being able to develop with Umbraco?

nul800sebastiaan commented 3 years ago

Sorry I missed this one, this looks like a regression we should look at!

OwainWilliams commented 3 years ago

Fixed in #9300

Thanks for raising the issue @skttl and for other reproducing it. Thanks @matthew-darby for fixing it.

h5yr

nul800sebastiaan commented 3 years ago

Cherry picked for 8.9.2 and merged up to 8.10.0 as well 👍