Revert "Fix inconsistent marker boundary order"

[pomek]

Reverts #19995

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 103685c. Configure here.}

[cursor]

Test uses nonexistent triggerBy API property silently ignored

Low Severity

The triggerBy property passed to elementToElement() does not exist in its config type (model | view | converterPriority are the only accepted keys) and is silently ignored. The converter is configured with model: 'simpleBlock' (a plain string), so normalizeModelElementConfig produces an empty attributes array, meaning this converter registers no reconversion triggers. The tests still pass only because the beforeEach block already registers a separate converter with model: { name: 'simpleBlock', attributes: ['toStyle', 'toClass'] } whose reduceChanges handler remains active. The triggerBy property is dead code that misleadingly suggests it controls reconversion behavior.

Additional Locations (1)

• packages/ckeditor5-engine/tests/conversion/downcasthelpers.js#L1295-L1298

 

^{Reviewed by Cursor Bugbot for commit 103685c. Configure here.}