Skip to content

#2440+#2006#14

Open
jamlung-ri wants to merge 5 commits intomainfrom
#2440+#2006
Open

#2440+#2006#14
jamlung-ri wants to merge 5 commits intomainfrom
#2440+#2006

Conversation

@jamlung-ri
Copy link
Copy Markdown
Member

AI-driven attempt at creating a basic Add to Collection dialogue and a basic Cascade Selector. It's currently only set up for one concept at a time, not a multi-select Add to Collection.

@snyaggarwal
Copy link
Copy Markdown
Contributor

snyaggarwal commented Apr 13, 2026

Findings:

  1. The "Add to Collection" button doesnt look like a clickable.
Screenshot 2026-04-13 at 16 48 57
  1. "Add to Collection" only appears when the Concept is opened. It should also appear in concept listing when selected.

  2. The dialog body for add to collection needs to have less padding

Screenshot 2026-04-13 at 16 47 16
  1. The Dialog needs to handle when the API returns 202 -- that means the Job is accepted and will be done in sometime. Currently it throws "Something went wrong"

  2. Added references color is out of OCL theme:

Screenshot 2026-04-13 at 16 51 37
  1. Added references message has redundant Concept ID (see above screenshot)

  2. Failed reference needs to be in "error" color

Screenshot 2026-04-13 at 16 52 53

@jamlung-ri
Copy link
Copy Markdown
Member Author

@snyaggarwal More updates made!

@snyaggarwal
Copy link
Copy Markdown
Contributor

snyaggarwal commented Apr 14, 2026

@jamlung-ri Findings:

  1. The formatting of body is still not right, too much padding and alignment needs to be better
Screenshot 2026-04-14 at 08 25 18
  1. Need to move "Close" button to top right as "X", same as other dialog

  2. Add to Collection in listing is individual concept add and not mullti, is this intentional?

Screenshot 2026-04-14 at 08 26 49
  1. Also all the labels needs to be extracted into language bundle en/translations.json

@jamlung-ri
Copy link
Copy Markdown
Member Author

@snyaggarwal More fixes are in, using the ocl-design-system repo as a reference!

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.

Cascade Component Add Reference to Collection (outside of Collection)

2 participants