Improve truncation and expand logic in sidebar list

[amadulhaxxani]

This pull request improves how truncation and expandability are handled in sidebar search list elements and their truncatable parts. The main focus is on ensuring that expandable states are tracked more accurately, especially when children elements are truncated, and simplifying the logic for truncation detection.

Sidebar search list expandability improvements:

• Added an initialTruncated flag to SidebarSearchListElementComponent to remember if any child was ever truncated, ensuring that expandability is preserved even after expansion.
• Updated the logic in updateExpandableState() to use both the current and initial truncation state, so the expandable button remains available if any content was ever truncated, even when expanded.
• Modified onTruncatedStateChange() to set initialTruncated when any child is truncated, improving consistency in expandable state tracking.

Truncatable part logic simplification:

• Removed redundant toggling of the expandable property in both toggle() and toggleWithoutId() methods of TruncatablePartComponent, as expandability is now managed externally. [1] [2]
• Simplified the truncation detection logic in truncateElement() by removing unnecessary child element checks and relying directly on scroll and offset height, making the code easier to maintain and understand. [1] [2]
• Changed the isExpanded getter to depend only on the expand property, decoupling it from the expandable state for clarity.

[Copilot]

Pull request overview

This PR improves truncation and expandability tracking in sidebar search list elements by introducing an initialTruncated flag to remember when content was ever truncated, ensuring expand/collapse buttons remain available even after expansion. It also simplifies truncation detection logic by removing unnecessary child element checks and decoupling the isExpanded getter from the expandable state.

Key Changes

• Added initialTruncated tracking to preserve expandability state after expansion
• Simplified truncation detection by relying solely on scroll/offset height comparison
• Modified expandable state management in components without IDs
• Updated test setup to use correct mock service IDs

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File	Description
src/app/shared/object-list/sidebar-search-list-element/sidebar-search-list-element.component.ts	Added initialTruncated flag and updated updateExpandableState() to preserve expandability after expansion
src/app/shared/truncatable/truncatable-part/truncatable-part.component.ts	Simplified truncateElement() logic, removed expandable toggling from toggle methods, changed toggleWithoutId() to set expandable state directly
src/app/shared/truncatable/truncatable-part/truncatable-part.component.spec.ts	Updated test to use correct mock service ID ('1' for collapsed state) and added comment explaining the change
yarn.lock	Updated multiple dependency versions (unrelated to feature changes)