Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Animate the cell by scaling it, just before it appears on screen #39

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vishalkevin11
Copy link

Feature :
Animate the cell by scaling it, just before it appears on screen

Feature Affected Areas :

  • Dailyfeeds List
  • Bookmarks List

Visual View of Feature :

cell_animation

…s on screen

Added for Dailyfeeds and Bookmark
@paulsumit1993
Copy link
Owner

This is quite an interesting change, but I would prefer not to merge this, as the animation is a little jarring and not fluid.

@paulsumit1993
Copy link
Owner

How about something subtle like the following?

    ///Animate the cell by scaling it, just before it appears on screen
    func collectionView(_ collectionView: UICollectionView, willDisplay cell: UICollectionViewCell, forItemAt indexPath: IndexPath) {
        if collectionView.isDragging {
            cell.transform = .init(scaleX: 0.95, y: 0.95)
            UIView.animate(withDuration: 0.5, delay: 0.0, options: [.curveEaseInOut], animations: {
                cell.transform = .identity
            })
        }
    }

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants