Alya Abbott
Alya Abbott
For me, it looked like @shuklamaneesh23 's screenshots just above (Chrome on a Mac).
Good questions! 1. I think the ideal behavior would require confirmation to change the API key, but probably let's leave that for a follow-up. 2. I like having the row...
Let's update the "refresh" icon to https://www.figma.com/design/msWyAJ8cnMHgOMPxi7BUvA/Zulip-Web-UI-kit?node-id=2027-43702&t=pedJe3BpVagOjfM1-0. I noticed that the stroke width doesn't match, and in general we're replacing all `fa` icons.
Otherwise looks good in light manual testing!
Oh, I see. That seems good, but not a blocker for merging this PR if it's non-trivial -- if I'm thinking about it right, it wouldn't come up too much.
> Another quirk, which seems like it should get its own issue, is the bot cards cut off the name of the bot owner very early: I think it's fine...
@Harsh2509 thanks for the work here! Please do not add review labels; this is managed by project maintainers, and by experienced contributors on their own PRs.
@pratik-pc Could you please do the first round of review on this one? I haven't tested it.
Closing in favor of more recent work in #31574. Thanks for the effort here, @Harsh2509!
@apoorvapendse Would it make sense for you to review this one once conflicts are resolved?