Alya Abbott
Alya Abbott
Yeah, we'll want to move towards the design [here](https://www.figma.com/design/msWyAJ8cnMHgOMPxi7BUvA/Zulip-Web-UI-kit?node-id=2072-9192&t=GSwQaYo3l4Vhrk2h-0). In the new design, I guess it's the same as the font for notes above the field, so we could use...
This is failing to handle improper inputs. Like changing the email to `commit@@@` looks like it goes through, but then appears this way when you reopen the modal: I would...
Ok, please add updated screenshots demonstrating all your changes to the PR description (and remove the outdated screenshot).
Please add dark theme screenshots, and screenshots of other user pills confirming that they weren't affected by your changes.
Thanks! Please carefully [self-review](https://zulip.readthedocs.io/en/latest/contributing/code-reviewing.html) your work, and fill out the self-review checklist (as applicable). I'm not sure why it was deleted from the description for this PR.
Can you compare with https://github.com/zulip/zulip/pull/36441 and comment on any differences between the approaches?
Ok, you can ping @sahil839 to take a look once this PR passes tests.
I confirmed that this looks in manual testing, so @sahil839 , please mark for integration review if the test looks good.
Thanks! The filter should be available when there are no resolved topics, not just when there are some.
Also, your screenshots don't match what I see when I test the PR, which has `Followed` capitalized (which is not expected). Please keep in mind that screenshots that don't match...