Skip to content

feat: group affected products in suggestions with tree view - #1257

Merged
fricklerhandwerk merged 1 commit into
NixOS:mainfrom
DarshanCode2005:group-affected-prod
Oct 12, 2026
Merged

fricklerhandwerk merged 1 commit into
NixOS:mainfrom
DarshanCode2005:group-affected-prod

Conversation

@DarshanCode2005

Copy link
Copy Markdown
Collaborator

Description:

  • Group long affected-product lists in suggestion detail views into a collapsible tree, addressing cases where dozens of related products (example: eap8-*) dominate the page (Group "Affected products" in suggestions #1253).
  • Add a frontend-only prefix-grouping heuristic: split product names on non-alphanumeric delimiters (-, ., /, …) and recursively group by the most common shared chunk (minimum group size 2).
  • Render the grouped structure with a new Ark UI TreeView wrapper, leaf rows still use the existing AffectedProduct component (full names, version badges, CPE tooltips unchanged).

@DarshanCode2005
DarshanCode2005 marked this pull request as draft August 24, 2026 11:01
@DarshanCode2005
DarshanCode2005 marked this pull request as ready for review October 1, 2026 06:06
@DarshanCode2005

Copy link
Copy Markdown
Collaborator Author
image image

@florentc
florentc force-pushed the group-affected-prod branch 2 times, most recently from c6f6815 to b77c2cb Compare October 8, 2026 15:26
@florentc

florentc commented Oct 8, 2026

Copy link
Copy Markdown
Member

Thanks a lot @DarshanCode2005. In order to get this ready faster I took the liberty to perform the rewrites and modifications I'd have requested.

  • Improved the overall feature
  • Improved the implementation of grouping
  • Made default collapsing not based on amount of children but whether a node has a descendant with an affected version
  • Added buttons to collapse all, expand all, or expand only affected versions
  • Improved readability and styling quirks
  • Added tests
image

Render affected products as a collapsible tree grouped by shared name-prefix chunks (example: eap8-*), using Ark UI TreeView

Co-authored-by: DarshanCode2005 <darshanthakaregsoc2023@gmail.com>
@florentc
florentc force-pushed the group-affected-prod branch from b77c2cb to ce6364d Compare October 8, 2026 16:08

@fricklerhandwerk fricklerhandwerk left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thanks @DarshanCode2005 and @florentc!

@fricklerhandwerk
fricklerhandwerk added this pull request to the merge queue Oct 12, 2026
Merged via the queue into NixOS:main with commit b4931e7 Oct 12, 2026
3 checks passed
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