Unnecessary Rebuilds

[Yogesh-Dubey-Ayesavi]

I just added to debugPrint('${widget.message.id} rebuilt'); to MessageView build method .

The logs i obtained then.

a0e30af0-daa0-46fa-a1f8-54af63a743ac.mp4

Each message even which are outside the viewPort are getting rebuilt, that causes huge memory consumption.

[Yogesh-Dubey-Ayesavi]

I figured out the problem, SingleChildScrollView is the responsible widget.

flutter_chatview/lib/src/widgets/chat_groupedlist_widget.dart

Line 161 in dc4a2f0

return SingleChildScrollView(

Since the _chatStreamBuilder method is embedded inside SingleChildScrollView so it builds each items at once, which in context with performance is not so good solution.

I tried removing SingleChildScrollView like this.

GestureDetector(
      onHorizontalDragUpdate: (details) => isEnableSwipeToSeeTime
          ? showPopUp
              ? null
              : _onHorizontalDrag(details)
          : null,
      onHorizontalDragEnd: (details) => isEnableSwipeToSeeTime
          ? showPopUp
              ? null
              : _animationController?.reverse()
          : null,
      onTap: widget.onChatListTap,
      child: Column(
        children: [
          Flexible(
            child: _animationController != null
                ? AnimatedBuilder(
                    animation: _animationController!,
                    builder: (context, child) {
                      return _chatStreamBuilder;
                    },
                  )
                : _chatStreamBuilder,
          ),
          TypingIndicator(
            typeIndicatorConfig: widget.typeIndicatorConfig,
            chatBubbleConfig: chatBubbleConfig?.inComingChatBubbleConfig,
            showIndicator: showTypingIndicator,
            profilePic: profileCircleConfig?.profileImageUrl,
          ),
          SizedBox(
            height: MediaQuery.of(context).size.width *
                (widget.replyMessage.message.isNotEmpty ? 0.3 : 0.14),
          ),
        ],
      ),
    );

flutter_chatview/lib/src/widgets/chat_groupedlist_widget.dart

Line 274 in dc4a2f0

physics: const NeverScrollableScrollPhysics(),

- physics: const NeverScrollableScrollPhysics(),  
+ controller: widget.scrollController,

flutter_chatview/lib/src/controller/chat_controller.dart

Line 92 in dc4a2f0

scrollController.position.minScrollExtent,

+scrollController.position.maxScrollExtent,

After making these changes guess what, it worked nicely.

Note

Using this method performance can be improved until an extent but, scrollingToReplyMessages will take a lot of time because GroupedListView is built on top of ListView.builders. I suggest using scrollable_positioned_list or scrollable_positioned_list_extended(with helper methods like scrollToMax extent and notification listeners) inside custom fork of GroupedList we can reach to threaded messages instantly.

[DhvanitVaghani]

@Yogesh-Dubey-Ayesavi Thanks for pointing out these issues. We had planned for this performance improvement, but you suggested some changes, I will try those and give feedback on this

[github-actions]

This issue is stale because it has been open for 30 days with no activity.

[github-actions]

This issue was closed because it has been inactive for 14 days since being marked as stale.