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

Scheduled post typing indicator and user typing indicator UI fix #8637

Open
wants to merge 4 commits into
base: feature_schedule_posts
Choose a base branch
from

Conversation

harshilsharma63
Copy link
Member

Summary

Fixed display of scheduled post indicator with user typing indicator

Ticket Link

Fixes https://mattermost.atlassian.net/browse/MM-62871

Checklist

  • Has UI changes

Device Information

iOS Emulator

Screenshots

both.mov
neither.mov
scheduled_post.mov
typing.mov

Release Note

None

@asaadmahmood
Copy link

asaadmahmood commented Feb 27, 2025

@harshilsharma63 looks good for the most part, can we reduce the spacing between the text and the icon to match this?
CleanShot 2025-02-27 at 13 54 36@2x

https://www.figma.com/design/VbEx15Tpvi9mXhs9IEjJtq/MM-45558-Drafts-%26-Schedule-Messages?node-id=5312-83916&t=1PgmCe2UrJvNTKaR-11

Also can we make the See all semibold

@harshilsharma63
Copy link
Member Author

can we reduce the spacing between the text and the icon to match this?

@asaadmahmood between the scheduled icon and "2 scheduled messages" text? If yes, the spacing is 12 units in both Figma and code. Not sure what to change.

Made "see all" semibold.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants