gini / gini-mobile-ios

Monorepo for Gini iOS SDKs
Other
6 stars 2 forks source link

PIA-4391_fix_delete_icon_interaction #394

Closed zladzeyka closed 1 year ago

zladzeyka commented 1 year ago
ValentinaIancu-Gini commented 1 year ago

@zladzeyka I think we also need to add the custom views inside the contentView in CaptureSuggestionsCollectionHeader

ValentinaIancu-Gini commented 1 year ago

Also, all the constraints should be added between custom views and the content view not self. We need to change this in all the cells where we modified how we are adding the custom views.

zladzeyka commented 1 year ago

@zladzeyka I think we also need to add the custom views inside the contentView in CaptureSuggestionsCollectionHeader

CaptureSuggestionsCollectionHeader : UICollectionReusableView : UIView -> It doesn't have a contentView

ValentinaIancu-Gini commented 1 year ago

@zladzeyka I think we also need to add the custom views inside the contentView in CaptureSuggestionsCollectionHeader

CaptureSuggestionsCollectionHeader : UICollectionReusableView : UIView -> It doesn't have a contentView

Pff I was checking Apple's documentation for UICollectionReusableView and I've scrolled down and saw a contentview but now I saw that is from UICollectionCell and not from UICollectionReusableView...sorry for the confusion.

zladzeyka commented 1 year ago

Also, all the constraints should be added between custom views and the content view not self. We need to change this in all the cells where we modified how we are adding the custom views.

Good point, I've updated it.

ValentinaIancu-Gini commented 1 year ago

@zladzeyka I fixed all the suggestions and comments :D.