JDK-8077916 : Complete development work for project one: make UI control skins public API
  • Type: Sub-task
  • Component: javafx
  • Sub-Component: controls
  • Priority: P3
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2015-04-16
  • Updated: 2021-04-15
  • Resolved: 2015-09-21
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 9
9Fixed
Related Reports
Blocks :  
JDK-8134716 - Remove use of internal classes methods from toys/Hello
Duplicate :  
JDK-8089138 - Remove use of internal classes methods from SceneBuilder
Relates :  
JDK-8136971 - Review the new TextPosInfo API
Relates :  
JDK-8265277 - SkinBase::registerChangeListener��� missing '@since 9' javadoc tag
Relates :  
JDK-8136404 - Review VirtualFlow API
Relates :  
JDK-8161082 - Key event consume() method doesn't work
Relates :  
JDK-8130354 - [ComboBox, DatePicker] Move editor and converter properties up to ComboBoxBase class
Description
Literally moving the classes from com.sun.javafx.scene.control.skin to javafx.scene.control.skin is trivial. The time consuming aspects are:

1) Reviewing each class and making only the minimal necessary API public. Guestimate: two weeks.
2) Performing API review - requiring time from Kevin, but also someone else such as Leif. Guestimate: two weeks.
3) There are some classes that are not quite skins, but not quite controls. We need to decide whether these remain private for now, or whether they graduate to fully-fledged controls. Guestimate: one week.
4) Updating 'the world (i.e. Scene Builder, etc)' to use the new APIs: one week.

Total guestimate (with no padding): six weeks.
Comments
Resolving as fixed now that the changesets have been pushed (we can file follow-on issues for further work).
21-09-2015
While at it, TextPosInfo is good for positioning caret, but not for getting the character under the cursor (JDK-8091012), which is useful e.g. for displaying a tooltip based on the word under the cursor. As I suggested in JDK-8136350, I think a reasonable API for TextPosInfo would be public OptionalInt getCharacterIndex(); public int getInsertionIndex(); Note the OptionalInt, to determine whether any character was hit at all.
11-09-2015
TextPosInfo needs to be changed to take character clusters into account, as noted in JDK-8092327, point 2. As it currently stands, TextPosInfo#getInsertionIndex() is broken: it may return an index inside a character cluster. Indeed, the skins themselves don't use TextPosInfo#getInsertionIndex(), but rather Utils#getHitInsertionIndex(TextPosInfo hit, String text), which is not available to the users, since Utils is in com.sun.javafx.scene.control.skin. Also note that Utils#getHitInsertionIndex in order to work around the limitation of TextPosInfo, needs to see the text again. I think the right thing to do is to make sure the returned TextPosInfo is correct, otherwise it is useless for anything but private implementation again. The most reasonable place to fix this seems to be to fix the hit test implementation on Text, rather than keep it broken and then work around it in text input controls.
11-09-2015
Changeset URL: http://hg.openjdk.java.net/openjfx/9-dev/rt/rev/5023330f0526
10-09-2015