This came out of the code review 
https://github.com/openjdk/jfx/pull/2302#issuecomment-6063971641

The RichEditorDemo uses image drop handler similar to that used in the Monkey 
Tester.  It is no surprise that I made the same mistake in not checking the 
editable flag.  One excuse I have is that in the demo it is possible neither to 
toggle the editable property, nor have a read-only model.  However, I would 
like to make this change for the same of people who would include parts of this 
project in their application.

Tests were not added since it's a demo.

---------
- [x] I confirm that I make this contribution in accordance with the [OpenJDK 
Interim AI Policy](https://openjdk.org/legal/ai).

-------------

Commit messages:
 - better
 - check editable flag

Changes: https://git.openjdk.org/jfx/pull/2341/files
  Webrev: https://webrevs.openjdk.org/?repo=jfx&pr=2341&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8393970
  Stats: 15 lines in 1 file changed: 13 ins; 0 del; 2 mod
  Patch: https://git.openjdk.org/jfx/pull/2341.diff
  Fetch: git fetch https://git.openjdk.org/jfx.git pull/2341/head:pull/2341

PR: https://git.openjdk.org/jfx/pull/2341

Reply via email to