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.

Sign up for GitHub

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

Jump to bottom

[Input Message] Fix overzealous animation #4291

Closed
macandcheese opened this issue Mar 23, 2022 · 2 comments
Closed

[Input Message] Fix overzealous animation #4291

macandcheese opened this issue Mar 23, 2022 · 2 comments
Assignees
@geospatialem @benelan
Labels
4 - verified Issues that have been released and confirmed resolved. bug Bug reports for broken functionality. Issues should include a reproduction of the bug.
Milestone
Sprint 05/09 - 05/20

Comments

@macandcheese
Copy link
Contributor

Actual Behavior

When making an input-message active by adding the attribute, currently the animation is rather ... in your face. I think we should tone this down a bit.

Screen.Recording.2022-03-23.at.12.15.47.PM.mov

bam-cooking-show

Expected Behavior

Don't use a scale-in effect to display the input message.

Reproduction Sample

https://codepen.io/mac_and_cheese/pen/ExoPEBe?editors=1111

Reproduction Steps

In codepen:
Click "add" in the slotted input button without entering anything in the input. See the wild and wacky animation.

Reproduction Version

80

Relevant Info

cc @mitc7862 @asangma

Regression?

No response
@macandcheese macandcheese added bug Bug reports for broken functionality. Issues should include a reproduction of the bug. 0 - new New issues that need assignment. needs triage Planning workflow - pending design/dev review. labels Mar 23, 2022
@benelan benelan added this to the Sprint 04/11 - 04/22 milestone Mar 25, 2022
@benelan benelan removed the needs triage Planning workflow - pending design/dev review. label Mar 25, 2022
@macandcheese macandcheese mentioned this issue Mar 29, 2022
Closed
@Elijbet Elijbet self-assigned this May 17, 2022
@Elijbet Elijbet added 1 - assigned Issues that are assigned to a sprint and a team member. and removed 0 - new New issues that need assignment. labels May 17, 2022
@Elijbet Elijbet mentioned this issue May 17, 2022
Merged
@jcfranco jcfranco added 3 - installed Issues that have been merged to master branch and are ready for final confirmation. and removed 1 - assigned Issues that are assigned to a sprint and a team member. labels May 19, 2022
@github-actions github-actions bot assigned benelan and geospatialem and unassigned Elijbet May 19, 2022
@github-actions
Copy link
Contributor

Installed and assigned for verification.

@geospatialem geospatialem added 4 - verified Issues that have been released and confirmed resolved. and removed 3 - installed Issues that have been merged to master branch and are ready for final confirmation. labels May 26, 2022
@geospatialem
Copy link
Member

Verified on next

verfication.mov

@nwhittaker nwhittaker mentioned this issue Jan 31, 2024
Closed
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@geospatialem geospatialem

@benelan benelan

Labels
4 - verified Issues that have been released and confirmed resolved. bug Bug reports for broken functionality. Issues should include a reproduction of the bug.
Projects
None yet
Milestone
Sprint 05/09 - 05/20
Development

No branches or pull requests
5 participants
@jcfranco @macandcheese @geospatialem @benelan @Elijbet