Mark UiDataProvider APIs as internal API

They're not meant for public consumption, but we need to keep them public as we have tests in a separate module (and can't move them in).
JetBrains · Aug 27, 2024 · 11b6486 · 11b6486
1 parent 1c6aa5c
commit 11b6486
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/...ridge/src/main/kotlin/org/jetbrains/jewel/bridge/actionSystem/RootDataProviderModifier.kt b/...ridge/src/main/kotlin/org/jetbrains/jewel/bridge/actionSystem/RootDataProviderModifier.kt
@@ -4,8 +4,10 @@ import androidx.compose.ui.node.ModifierNodeElement
 import com.intellij.openapi.actionSystem.DataSink
 import com.intellij.openapi.actionSystem.UiDataProvider
 import org.jetbrains.annotations.VisibleForTesting
+import org.jetbrains.jewel.foundation.InternalJewelApi
 
 @VisibleForTesting
+@InternalJewelApi
 public class RootDataProviderModifier : ModifierNodeElement<RootDataProviderNode>(), UiDataProvider {
     private val rootNode = RootDataProviderNode()
 

diff --git a/...af-bridge/src/main/kotlin/org/jetbrains/jewel/bridge/actionSystem/RootDataProviderNode.kt b/...af-bridge/src/main/kotlin/org/jetbrains/jewel/bridge/actionSystem/RootDataProviderNode.kt
@@ -5,16 +5,21 @@ import androidx.compose.ui.node.TraversableNode
 import androidx.compose.ui.node.traverseDescendants
 import com.intellij.openapi.actionSystem.DataSink
 import com.intellij.openapi.actionSystem.UiDataProvider
+import org.jetbrains.annotations.VisibleForTesting
+import org.jetbrains.jewel.foundation.InternalJewelApi
 import org.jetbrains.jewel.foundation.actionSystem.DataProviderNode
 
+@VisibleForTesting
+@InternalJewelApi
 public class RootDataProviderNode : Modifier.Node(), UiDataProvider {
     override fun uiDataSnapshot(sink: DataSink) {
         val context = DataProviderDataSinkContext(sink)
 
         traverseDescendants(DataProviderNode) { dp ->
             if (dp is DataProviderNode) {
                 if (!dp.hasFocus) {
-                    return@traverseDescendants TraversableNode.Companion.TraverseDescendantsAction.SkipSubtreeAndContinueTraversal
+                    return@traverseDescendants TraversableNode.Companion.TraverseDescendantsAction
+                        .SkipSubtreeAndContinueTraversal
                 } else {
                     dp.dataProvider(context)
                 }