Skip to content

ui: Distinguish isBot: true users in more places, not just message senders in the message list #636

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
chrisbobbe opened this issue Apr 26, 2024 · 3 comments

Comments

@chrisbobbe
Copy link
Collaborator

chrisbobbe commented Apr 26, 2024

Quoting myself in #605 (comment) , which added bot markers on message senders in the message list:

After this, there are still some places in the UI where we show the name of a bot user without making it clear it's a bot. For example, it would probably be helpful to show the bot icon next to a bot user's name on the profile page (ProfilePage). I'm not sure if we need to show the icon in all the places where names appear, though. [Let's make] a checklist of all the places where bots' names might be displayed in the app, indicating which ones should start marking bots with the bot icon, which ones don't need to do that, and which ones [we're] not sure about[.]

This is a followup to:

@chrisbobbe chrisbobbe added this to the Launch milestone Apr 26, 2024
@chrisbobbe
Copy link
Collaborator Author

@sm-sayedi, are you still up for drafting a checklist like this? 🙂

@sm-sayedi
Copy link
Collaborator

@sm-sayedi, are you still up for drafting a checklist like this? 🙂

Unfortunately, I haven't completed the list yet as I got busy with #608! After I finish the first round of tests for that, I will return to this one and finalize the list. Should I then post it as a comment under this issue?

@chrisbobbe
Copy link
Collaborator Author

Sure, sounds good! Thank you!

@gnprice gnprice modified the milestones: Launch, B2: Summer 2024 May 9, 2024
sm-sayedi added a commit to sm-sayedi/zulip-flutter that referenced this issue Jun 26, 2024
sm-sayedi added a commit to sm-sayedi/zulip-flutter that referenced this issue Jun 26, 2024
sm-sayedi added a commit to sm-sayedi/zulip-flutter that referenced this issue Jun 27, 2024
sm-sayedi added a commit to sm-sayedi/zulip-flutter that referenced this issue Jun 27, 2024
sm-sayedi added a commit to sm-sayedi/zulip-flutter that referenced this issue Jul 3, 2024
@sm-sayedi sm-sayedi removed their assignment Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging a pull request may close this issue.

3 participants