Skip to content

fix: spacing between message in MessageList#3429

Merged
khushal87 merged 4 commits intodevelopfrom
fix/spacing-between-messages
Feb 25, 2026
Merged

fix: spacing between message in MessageList#3429
khushal87 merged 4 commits intodevelopfrom
fix/spacing-between-messages

Conversation

@khushal87
Copy link
Member

This pull request introduces several improvements and refactors to the message components, focusing on simplifying message grouping styles, cleaning up the MessageFooter logic, and updating the pinned message header. The changes enhance maintainability, improve style handling, and remove unnecessary complexity from the codebase.

Message Grouping and Styling Refactor:

  • Refactored MessageSimple component to use a new useStyles hook for dynamic style calculation based on message grouping and alignment, replacing static style objects and simplifying style logic. [1] [2] [3]
  • Removed unused style properties and consolidated style handling in MessageSimple, eliminating redundant container and grouping styles.

Message Footer Logic Simplification:

  • Removed the OnlyVisibleToYouComponent and related props (otherAttachments, deletedMessagesVisibilityType, isDeleted) from MessageFooter, streamlining its logic and prop interface. [1] [2] [3] [4] [5] [6] [7] [8]
  • Updated MessageDeleted to no longer pass the isDeleted prop to MessageFooter, reflecting the simplified footer logic.

Pinned Message Header Updates:

  • Renamed and moved MessagePinnedHeader to a new directory, updated icon size, and adjusted font weight for improved appearance and code organization. [1] [2] [3]
  • Fixed import paths for MessagePinnedHeader in Channel.tsx. [1] [2]

Equality Checks Optimization:

  • Simplified equality checks for groupStyles in both Message and MessageContent components, improving performance and reducing unnecessary renders. [1] [2]

Minor UI Adjustments:

  • Adjusted padding and map view size in MessageLocation for better visual consistency.
  • Removed top margin from message wrapper in MessageWithContext for improved layout.

@khushal87 khushal87 requested a review from isekovanic February 25, 2026 13:58
@Stream-SDK-Bot
Copy link
Contributor

SDK Size

title develop branch diff status
js_bundle_size 424 KB 424 KB +14 B 🟢

@khushal87 khushal87 merged commit 2a51aa3 into develop Feb 25, 2026
5 checks passed
@khushal87 khushal87 deleted the fix/spacing-between-messages branch February 25, 2026 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants