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

NMC/2341 - NCTrashView #95

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

Conversation

TSI-shwetawaikar
Copy link

In this PR:
We have added some changes related to Grid View selection in "Deleted files" section in More Tab.

@TSI-shwetawaikar TSI-shwetawaikar changed the title nmc/2341 NCTrash Grid cell selection changes nmc/2341 NCTrashView Jun 28, 2023
@TSI-shwetawaikar TSI-shwetawaikar added custom MagentaCLOUD customisation approved labels Jun 28, 2023
@@ -84,7 +84,7 @@ extension NCTrash: UICollectionViewDataSource {
cell = listCell
} else {
// GRID
guard let gridCell = collectionView.dequeueReusableCell(withReuseIdentifier: "gridCell", for: indexPath) as? NCGridCell else { return UICollectionViewCell() }
guard let gridCell = collectionView.dequeueReusableCell(withReuseIdentifier: "gridCell", for: indexPath) as? NCTrashGridCell else { return UICollectionViewCell() }

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestion: Wherever existing code is customized. Put a comment above mentioning "MagentaCloud Customization" to identify our customization during code conflict. It will also help new developers.

@TSI-amrutwaghmare TSI-amrutwaghmare force-pushed the nmc/2341-Trash-view branch 2 times, most recently from 2e1dab3 to 8f09012 Compare July 19, 2023 12:47
@TSI-amrutwaghmare TSI-amrutwaghmare changed the title nmc/2341 NCTrashView NMC/2341 - NCTrashView Oct 18, 2023
@TSI-amrutwaghmare TSI-amrutwaghmare force-pushed the nmc/2341-Trash-view branch 2 times, most recently from aed7af8 to 8336512 Compare March 19, 2024 06:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved custom MagentaCLOUD customisation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants