Embedding: do not lock controls, always commit on new data

[PrimozGodec]

Issue

• Widget is locking controls while embedding. It is not required since we can easily cancel embedding when control is changed.
• The widget does not embed on new data when auto-apply is off - it is different than in other widgets (e.g. distances)

Description of changes

• Widget changed to not lock controls anymore
• Widget changed to always commit on new data

Includes

• [X] Code changes
• [X] Tests
• [ ] Documentation

[ajdapretnar]

I've tried it and it looks ok. There's one thing that I'm unsure of. When I select a dropdown, but I don't change the attribute, the computation is always restarted. Seems unnecessary, but perhaps it is how Qt works.

[codecov[bot]]

Codecov Report

Merging #182 into master will increase coverage by 0.11%. The diff coverage is 88.23%.

@@            Coverage Diff             @@
##           master     #182      +/-   ##
==========================================
+ Coverage   66.36%   66.47%   +0.11%     
==========================================
  Files          13       13              
  Lines        2997     3001       +4     
  Branches      439      439              
==========================================
+ Hits         1989     1995       +6     
+ Misses        890      888       -2     
  Partials      118      118              

[PrimozGodec]

I've tried it and it looks ok. There's one thing that I'm unsure of. When I select a dropdown, but I don't change the attribute, the computation is always restarted. Seems unnecessary, but perhaps it is how Qt works.

Yes, it is how Qt works. Anyway, I added a workaround since recomputation in this widget is quite expensive.