-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Padding seems off for the latest message with the typing indicator moved to the top #9251
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
Comments
Triggered auto assignment to @mountiny ( |
Triggered auto assignment to @arielgreen ( |
I could find that this is related to two PRs recently
I dont think this needs to be a deploy blocker since it is mainly a small styling problem. |
Sorry @arielgreen for the ping, I will actually wait to first refine how it should look like exactly and all the expected behaviours as there seems to be some discussion about it in the Slack thread. |
Making this weekly for now, we need to find some consensus on what would be the best way to render these.
|
Uh oh!
There was an error while loading. Please reload this page.
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Action Performed:
Expected Result:
We should have the metadata above and below the compose box as before and make sure it is vertically aligned. Decided in Slack here.
Actual Result:
Padding seems too high for the latest message with the typing indicator moved to the top
Workaround:
unknown
Platform:
Where is this issue occurring?
Version Number: 1.1.69-2
Reproducible in staging?: Y
Reproducible in production?: N
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by: @AndrewGable
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1653937814767159
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: