Week 3: Reviewer Feedback and Fixes

Wait 5 sec.

This is a weekly update from my Google Summer of Code 2026 project withKDE, improving effect widgets in Kdenlive, a free and open sourcevideo editor.This week was driven entirely by reviewer feedback onMR !887; the draft MR for the Curves Widget.Point snapping instead of rejectionWhen two curve control points are placed too close on the x-axis,avfilter/MLT crashes with Key point coordinates are too close or not strictly increasing.The original guard in AssetParameterModel::internalSetParameter silentlyrejected the update, the user moved a point but nothing happened visibly.Reviewer Bernd Jordan flagged this as confusing UX.The fix: instead of rejecting, snap the offending point so it maintainsthe minimum safe x-distance (~0.00266) from its neighbor. The curve isalways valid, always sent to MLT, and the user sees immediate feedback.Removing the 5 point limit for avfilter.curvesThe curve editor had a maximum of 5 control points, inherited fromfrei0r.curves. JB pointed out there is no reason to keep this limitfor avfilter.curves, it is a frei0r-specific constraint.The fix: setMaxPoints is now only called for frei0r.curves. Theavfilter.curves widget has no upper limit on control points.Both fixes are inMR !887.JB also noted the null placeholder approach in m_widgets for secondaryav_curve params is not ideal long term, waiting on his direction beforetouching that.