Skip to content
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

Update search.mdx #3667

Merged
merged 13 commits into from
Feb 2, 2024
Merged

Update search.mdx #3667

merged 13 commits into from
Feb 2, 2024

Conversation

oliverHPE
Copy link
Collaborator

Have updated several sections of the search guidance to provide more accurate info.

Information updated includes:

Placeholder text
Icon position
Clear seach icon
Accessibility

The placeholder text on the 'don't' guidance needs to include: "Search first name, last name, email, role, location"and if possible is long enough to show truncation happening.

For the clear search icon, we have two examples we can show. One from Chrome and one from Safari. They're in the Figma file.

Thanks

Have updated several sections of the form to improve guidance and provide more accurate info.

Information updated includes:

Placeholder text
Icon position
Clear seach icon
Accessibility

The placeholder text on the 'don't' guidance needs to include: "Search first name, last name, email, role, location"and if possible is long enough to show truncation happening.

For the clear search icon, we have two examples we can show. One from Chrome and one from Safari. They're in the Figma file.

Thanks
@oliverHPE oliverHPE requested review from taysea and jcfilben January 30, 2024 10:57
@taysea
Copy link
Collaborator

taysea commented Jan 30, 2024

Thanks, Oliver! I've made this PR with some suggestions to wording/adjusting examples.

The one thing I wasn't clear on was the rationale for when you would have the "Search" icon at the far right of the input. The rationale for having it before the placeholder text was clear, but feels like we have a gap for the other position. Can you add some content for that?

@oliverHPE
Copy link
Collaborator Author

Thanks, Oliver! I've made this PR with some suggestions to wording/adjusting examples.

The one thing I wasn't clear on was the rationale for when you would have the "Search" icon at the far right of the input. The rationale for having it before the placeholder text was clear, but feels like we have a gap for the other position. Can you add some content for that?

Okay I see, thanks Taylor. I didn't want to specify whether to use the icon on the left or right as there could be reasons to use either or. Also we don't seem to have a default preference. The point I was trying to get across is that the icon should be seen before the label is seen. Whether it's on the left or right, the icon should be seen first to help with scanning the page. If the icon is seen second, then it's more decoration and secondary support for the label rather than a primary way of locating the search input. I need to adjust the wording to make that more clear if you didn't pick that up. I think the word 'before' here could be the problem, as it seems like it is perhaps suggesting the icon 'before the label' means should be to the left of the label (in left to right reading languages). Saying the icon should be seen first is probably more clear and isn't recommending left or right.

@taysea taysea requested review from taysea and jcfilben February 1, 2024 17:06
Copy link
Collaborator

@taysea taysea left a comment

Choose a reason for hiding this comment

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

LGTM, thanks Oliver for writing the content.

Copy link
Collaborator

@jcfilben jcfilben left a comment

Choose a reason for hiding this comment

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

We talk about icon positioning in both the Search Icon/Placement section and the Variants/Icon position.

I think we should just have this information in one of these sections so we don't duplicate content.

Other than that everything else looks good!

@taysea
Copy link
Collaborator

taysea commented Feb 2, 2024

We talk about icon positioning in both the Search Icon/Placement section and the Variants/Icon position.

I think we should just have this information in one of these sections so we don't duplicate content.

I incorporated this feedback! @jcfilben

@taysea taysea requested a review from jcfilben February 2, 2024 00:06
Copy link
Collaborator

@jcfilben jcfilben left a comment

Choose a reason for hiding this comment

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

Looks good!

@taysea taysea merged commit b4c9e43 into master Feb 2, 2024
7 of 9 checks passed
@taysea taysea deleted the SearchHPE-patch-1 branch February 2, 2024 17:11
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