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

Add animation when sorting attendee list #3981

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

anilsonmez-simsoft
Copy link
Contributor

@anilsonmez-simsoft anilsonmez-simsoft commented Mar 2, 2022

Attendees table is sorted with an animation according to the attendee role changes.

Closes AB#340

User changes

  • Users get visual feedback while the order of attendees table changing according to the role change

Super User changes

  • None

Admin changes

  • None

System admin changes

  • anet.yml or anet-dictionary.yml needs change
  • db needs migration
  • documentation has changed
  • graphql schema has changed

Checklist

  • Described the user behavior in PR body
  • Referenced/updated all related issues
  • commits follow a repo#issue: Title title format and these 7 rules
  • commits have a clean history, otherwise PR may be squash-merged
  • Added and/or updated unit tests
  • Added and/or updated e2e tests
  • Added and/or updated data migrations
  • Updated documentation
  • Resolved all build errors and warnings
  • Opened debt issues for anything not resolved here

@anilsonmez-simsoft anilsonmez-simsoft self-assigned this Mar 2, 2022
@anilsonmez-simsoft anilsonmez-simsoft force-pushed the AB-340-reorder-attendee-list-animation branch 2 times, most recently from bc00a4a to 1616706 Compare March 3, 2022 09:35
@anilsonmez-simsoft anilsonmez-simsoft force-pushed the AB-340-reorder-attendee-list-animation branch from 1616706 to 631de2c Compare March 3, 2022 09:37
@anilsonmez-simsoft anilsonmez-simsoft changed the title AB#340: Add animation when sorting attendee list Add animation when sorting attendee list Mar 3, 2022
@anilsonmez-simsoft
Copy link
Contributor Author

This can be closed as we agreed on another solution here #3983

@gjvoosten gjvoosten marked this pull request as draft February 16, 2023 12:18
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.

1 participant