appflowy-editor icon indicating copy to clipboard operation
appflowy-editor copied to clipboard

[Bug] Type error from TableBlockKeys.colsHeight int being used where double? is expected

Open mo-edumo opened this issue 2 months ago • 0 comments

Bug Description

I see various type error issues related to a table colsHeight holding an int and being used where a double is expected.

I would love to help, but unfortunately do not have time. So, I hope it is useful to just report what I see in the error reporting service I use.

_TableColBorderState.buildFixedBorder.<T> (table_col_border.dart:89) Container.height seem to expect a double and complains when getting an int from TableBlockKeys.colsHeight

  Container buildFixedBorder(BuildContext context) {
    return Container(
      width: widget.tableNode.config.borderWidth,
      height: context.select(
        (Node n) => n.attributes[TableBlockKeys.colsHeight],
      ),
      color: Colors.grey,
    );
  }

_TableColBorderState.buildResizableBorder.<T> (table_col_border.dart:75) Same issue with Container.height expecting a double

        },
        child: Container(
          key: _borderKey,
          width: widget.tableNode.config.borderWidth,
          height: context.select(
            (Node n) => n.attributes[TableBlockKeys.colsHeight],
          ),
          color: _borderHovering || _borderDragging
              ? widget.borderHoverColor
              : widget.borderColor,
        ),

How to Reproduce

I'm unfortunately not sure what users have done, but I currently see the issue reported only on Android. I primarily test on web currently and have not experienced the issue or caught errors in the console logs.

Expected Behavior

I would expect that the editor component doesn't throw errors like this.

Operating System

Android

AppFlowy Editor Version(s)

6.1.0

Screenshots

No response

Additional Context

I use Flutter 3.35.

mo-edumo avatar Sep 23 '25 08:09 mo-edumo