Fix: hide repeated avatar and username for grouped messages#82
Fix: hide repeated avatar and username for grouped messages#82beezly wants to merge 3 commits intoviktorstrate:mainfrom
Conversation
|
Nice fix, I've thought about doing it as well but haven't come around to it yet. When testing it though I can see that the hover actions are cut off for the row that it's inside. This is not a problem when the display name is shown since it is a part of the same row. I don't know if there is an option to allow drawing outside the rect of the row. 🤔 
|
|
Nice find! I think the only good fix for this is going to be to float a popover. I'll work on it. Marking this as a draft for now. |
NSHostingView inside NSTableView with usesAutomaticRowHeights enters an infinite layout loop: layout() flushes SwiftUI transactions, which invalidate constraints, which re-enter layout. AppKit detects this and crashes with NSGenericException. Replace the entire auto-sizing mechanism with SelfSizingHostingView: - sizingOptions = [] prevents the hosting view from participating in Auto Layout constraint solving. - invalidateIntrinsicContentSize() is overridden to NOT call super (which would re-enter the constraint system) but instead schedule a coalesced async height update via DispatchQueue.main.async. - measureHeight() uses a temporary NSHostingController.sizeThatFits() to properly measure content at the cell width, including text wrapping. - heightOfRow returns cached heights (default 60px). noteHeightOfRows is called with animation duration 0 to avoid visible row resizing. Also removes the old measurementHostingView and handleTableResize.
…same sender Only show the profile header (avatar + username) on the first message in a consecutive group from the same sender. Timestamps remain visible on every message.
beezly
force-pushed
the
fix/group-sender-messages
branch
3 times, most recently
from
51b7298 to
0a392fa
Compare
Floating panel with quick reactions (👍🎉❤️), reply, reply in thread, and pin actions. Panel is dismissed when scrolling or resizing, and hidden when the row top is outside the visible timeline area.
beezly
force-pushed
the
fix/group-sender-messages
branch
from
0a392fa to
d239d3d
Compare
|
Rebased onto #85. Hover actions are now a floating NSPanel (HoverActionsPanel) instead of the inline ZStack |
2 participants
Summary
When multiple consecutive messages are from the same sender, the avatar and username are now only shown on the first message in the group. Timestamps remain visible on every message.
Before
After
Changes
shouldIncludeProfileHeader(at:)helper toTimelineViewControllerthat checks whether the previous visible item is a message from the same senderTimelineItemRowViewnow accepts anincludeProfileHeaderparameter (defaults totrue)