Looks good!

Some feedback:

• When typing an email or group, can we maintain the tab to create email pill?
• Highlight the search box when it's active. See what's in production today:

• Maintain the same font size as what's in production:

• In the dropdown, display avatar for each item (also, if a member is a guest, display the guest pil)
• Make sure that the dropdown box width matches the width of the search box

Looks good!

Some feedback:

1. When typing an email or group, can we maintain the tab to create email pill?
2. Highlight the search box when it's active. See what's in production today:

3. Maintain the same font size as what's in production:

vs
4. In the dropdown, display avatar for each item (also, if a member is a guest, display the guest pil)
5. Make sure that the dropdown box width matches the width of the search box

fyi: This PR is still in draft.

Latest demo:

This is ready for UXQA review @ericokuma

Looks great!
Some feedback:

• Make sure that guest users have a separate section in the dropdown:

  my-rill-project overview - Rill - 3 June 2025

• The placeholder text is way too small. I believe they should be the same font size as user names

• Update placeholder text to say: "Search users, groups, or add emails, separated by commas"

• When there are more than 1 row of pills, the dropdown doesn't move along with the bottom edge of the search field

  my-rill-project overview - Rill - 3 June 2025

This is ready for code review @ericpgreen2

Ah dang…I guess the placeholder text is now too long with the font size change. Sorry to do this but let's revert back to what you had before

otherwise LGTM! @jkhwu?

• I guess another thing I noticed is the search field height up larger than the invite button.
  
  Here's what it looked like before:

• And another thing I noticed is when the search field is blank, the role selector shouldn't be there:

NIT: https://www.loom.com/share/9d4fa8d95bd841b3bf304c092e56bbcd?sid=2039f359-7555-448b-b3f9-fa07d13592b5

NIT: Admin role isn't right-aligned with the other roles

my-rill-project overview - Rill - 4 June 2025

• Actually, General Access area should be fixed and everything scrolls underneath it. That might solve this

@jkhwu UXQA:

• There should be hover states for the following:
  • Link to docs
  • Dropdown selector for the general access (invite only or everyone at) selector

NIT: Admin role isn't right-aligned with the other roles

How do I replicate that? Here's what I'm seeing:

• Dropdown selector for the general access (invite only or everyone at) selector

What is the hover state for this? Can you link it or provide a screenshot?

Update: Found it

Addressed the feedback above @ericokuma

From @jkhwu, the last item is that the "invite" button should be enabled once a user starts typing something into the search:

in Prod:

in this PR:

Otherwise, LGTM!!

From @jkhwu, the last item is that the "invite" button should be enabled once a user starts typing something into the search:in Prod:

in this PR:

We shouldn't do that. The invite button should only be available when a chip/pill is created from the users' input. We want to avoid submitting users' jibberish texts.

While that's true, you could enter an actual email: janet.hwu@rilldata.com but if you didn't press the tab, comma, or space key, the invite button remains disabled. So it looks like it's broken

While that's true, you could enter an actual email: janet.hwu@rilldata.com but if you didn't press the tab, comma, or space key, the invite button remains disabled. So it looks like it's broken

Ok, resolved!

hmmm, the UX is not quite right. Could you reference what you've already built in production?

1. Hitting invite triggers a loading state in the invite button
2. Hitting invite doesn't first convert the string into pill
3. Interestingly, there's some complex logic you built previously with regards to non-pill strings and the enablement/disablement of the invite button