[BUG] - In SuperEditor it's not straightforward to theme caret/handles on mobile according to brightness

[miguelcmedeiros]

Package Version
super_editor main branch (c3190a9cc77b25d9ce18c798b1b7cdcecab13ecc)

User Info
Superlist

Details
The only way I found to consistently customize the color of caret and handles on mobile so that it changes when the brightness changes is to wrap SuperEditor with a Theme:

Theme(
  data: Theme.of(context).copyWith(
    primaryColor:
      Theme.of(context).brightness == Brightness.light
        ? Colors.black
        : Colors.white,
  ),
  child: SuperEditor(...),

Platform
iOS and Android

Flutter version
Flutter master

[miguelcmedeiros]

Cc @matthew-carroll

[angelosilvestre]

It is possible to configure SuperEditorIosHandlesDocumentLayerBuilder and SuperEditorAndroidHandlesDocumentLayerBuilder with the desired caret color, but we have a bug where it's not immediately being updated when switching the brightness.

@matthew-carroll Could you take a look at this comment to see if we can move forward?

[angelosilvestre]

@miguelcmedeiros

Now that #2388 is merged you should be able to set the desired colors in SuperEditorIosHandlesDocumentLayerBuilder and SuperEditorAndroidHandlesDocumentLayerBuilder. For example:

SuperEditor(
  documentOverlayBuilders: [
    DefaultCaretOverlayBuilder(
      caretStyle: const CaretStyle().copyWith(color: isLight ? Colors.black : Colors.redAccent),
    ),
    if (defaultTargetPlatform == TargetPlatform.iOS) ...[
      SuperEditorIosHandlesDocumentLayerBuilder(
        handleColor: isLight ? Colors.blue : Colors.redAccent,
      ),
      SuperEditorIosToolbarFocalPointDocumentLayerBuilder(),
    ],
    if (defaultTargetPlatform == TargetPlatform.android) ...[
      SuperEditorAndroidToolbarFocalPointDocumentLayerBuilder(),
      SuperEditorAndroidHandlesDocumentLayerBuilder(
        caretColor: isLight ? Colors.blue : Colors.redAccent,
      ),
    ],
  ],
)

Simulator.Screen.Recording.-.iPhone.15.Plus.-.2024-12-31.at.09.42.26.mp4

[miguelcmedeiros]

Thanks @angelosilvestre. Feel free to close the issue.