Skip to content

Update knowledge seed examples, keep context sticky #719

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jeff-phillips-18
Copy link
Collaborator

Closes #699

Description

Updates the Create seed data page in the Knowledge contribution wizard:

  • Uses expandable cards for each seed example
  • Updates the mechanism for selecting context from a file
  • Adds actions to clear the seed data or edit the context
  • Make the context sticky when expanded
  • Adds word counts for the questions and answers

Screen shots

image

image

image

image

image

image

image

@Misjohns
Copy link
Collaborator

Misjohns commented Apr 10, 2025

@jeff-phillips-18 Below are my comments:

See example cards
On the Seed example cards the text gets truncated very early. The text should expand to right margin of card.
image

Currently, the text boxes do not adjust to accommodate the text which cuts off content. Use PF autoResize text box to modify height based on contents. This would grow the box as the user types so they can see anything they enter but still provides them the ability to manually adjust the height of the text boxes in desired.
image

For the helper text, we decided to only show the # of words the associated text box container (ie. 35 words), not 35/150 words. The text field should only be marked on error if the user goes above the allotted word count. At that time we could display "0 words remaining". If we need to also display a word count then something like, "35 words / 5 words above limit" so user can easy determine the number of words they need to remove from text box. All empty text boxes in that seed example that have not been filled would display "0 words remaining" error. LMK if this is not feasible at this time. Alternatively, we could prevent the user from typing anything beyond the maximum word limit but this is not the ideal experience. @kaedward Please assist with microcopy for error message: "35 words / 5 words above limit".
image

When a seed example card is expanded all 3 QnA pairs should appear in card.
image

@jeff-phillips-18
Copy link
Collaborator Author

jeff-phillips-18 commented Apr 10, 2025

@jeff-phillips-18 Below are my comments:

See example cards On the Seed example cards the text gets truncated very early. The text should expand to right

That is only because in this example there is a newline in the copy. Here is an instance where there isn't:

image

Currently, the text boxes do not adjust to accommodate the text which cuts off content. Use [PF autoResize text box]

👍

For the helper text, we decided to only show the # of words the associated text box container (ie. 35 words), not 35/150 words. The text field should only be marked on error if the user goes above the allotted word count. At that time we could display "0 words remaining". If we need to also display a word count then something like, "35 words / 5 words above limit" so user can easy determine the number of words they need to remove from text box. All empty text boxes in that seed example that have not been filled would display "0 words remaining" error. LMK if this is not feasible at this time. Alternatively, we could prevent the user from typing anything beyond the maximum word limit but this is not the ideal experience. @kaedward Please assist with microcopy for error message: "35 words / 5 words above limit".

Yeah, I put that there so the user knows where they are over and by how much. Happy to change it however you like.

When a seed example card is expanded all 3 QnA pairs should appear in card.

This is not possible and I thought the point was to keep the context visible when entering all the answers. If we don't scroll the question and answer area we get:

image

You can see that by the time the user gets to Q and A 3, the context will be long gone

@kaedward
Copy link

@Misjohns for the "above word limit" alert, I recommend "100/99 words". The text format shouldn't change, the indicator that something is wrong is the red icon.

if we're following the pattern, the 3rd text box in the screenshot should have the help text: "0/0 words". The first help text should be "31/[number remaining] words"
image

Is this pattern overly complicated? I admit that I don't know exactly how editing a previous context would affect the following ones if you have already filled out all fields. I've never encountered this type of issue where word limit is shared between multiple fields.


I also advise against using the tooltip to show the full context that has been truncated. This might work if the full context was fewer than ~150 characters, but since the contexts can be up to 500 words, I wouldn't use the tooltip at all, even when the context is short.

Why is the context truncated when the card is expanded? I would expect the full context to be shown, instead of having an additional dropdown option to reveal the full text.

@Misjohns
Copy link
Collaborator

@kaedward The context area of the card is sticky and remains visible until the user scrolls to the bottom of the 3rd QnA for that context. By default the full context displays but we wanted the user to be able to gain some more real estate to enter their QnA pairs.

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.

[UX] Improve Seed example creation design and flow
3 participants