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

Folders and Files tinting & filtering customizations #65

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

Conversation

surinder-tsys
Copy link

@surinder-tsys surinder-tsys commented Apr 12, 2023

NMC-1922: Filtering only folders and showing them as per NMC-2893 Task
Update UnifiedSearchFragment.kt.
Commit id: 4fb3fb3 from branch bug/NMC-1652.

NMC-3192: ExoPlayer seek forward and backward customization.

NMC-2140: Tinting removed for folder and file icons and not overlaying icon for folders.
Share icon changed.

@surinder-tsys surinder-tsys added the custom MagentaCLOUD customisation label Apr 12, 2023
@tobiasKaminsky
Copy link

Same here, can you outline purpose of it? :)

@surinder-tsys
Copy link
Author

Same here, can you outline purpose of it? :)

This PR contains the following:

  1. Customization related to Grid items.
  2. Fixes related to search view.
  3. Copy/move functionality.
  4. Fixes for duplicate items in Favorite section.
  5. Camera rationale permission handling.

@surinder-tsys
Copy link
Author

@tobiasKaminsky
Some of the changes can be upstreamed.
I will analyze those and will create upstream PR.

@surinder-tsys
Copy link
Author

@tobiasKaminsky I have extracted the fixes which can be up-streamed and here are the links:

  1. Media view refresh infinite loop nextcloud/android#11521
  2. Camera permission rationale nextcloud/android#11520
  3. Favourite shown twice nextcloud/android#11519
  4. User check added when sync folder runs after user clears the data. nextcloud/android#11518
  5. Avoid search collapse after page refresh nextcloud/android#11522

Can you please go through them so that I can remove the up-streamed code from this PR.

@tobiasKaminsky
Copy link

Remaining parts do not seem to be Magenta specific, so they can split up and also be upstreamed.

@@ -251,6 +252,7 @@ class UnifiedSearchFragment : Fragment(), Injectable, UnifiedSearchListInterface
}

override fun onQueryTextSubmit(query: String): Boolean {
KeyboardUtils.hideKeyboardFrom(requireContext(), binding.root)
Copy link
Author

Choose a reason for hiding this comment

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

@tobiasKaminsky This logic will hide the keyboard every time whenever user do search.
Let me know if this also has to be up-streamed.

@surinder-tsys surinder-tsys force-pushed the nmc/1922-files branch 2 times, most recently from 6392b4f to 98c5214 Compare August 8, 2024 14:18
@surinder-tsys surinder-tsys changed the title Nmc/1922 Files Folders and Files tinting and filtering customizations Sep 3, 2024
@surinder-tsys surinder-tsys changed the title Folders and Files tinting and filtering customizations Folders and Files tinting & filtering customizations Sep 4, 2024
Update UnifiedSearchFragment.kt.
Commit id: 4fb3fb3 from branch bug/NMC-1652.
NMC-2140: Tinting removed for folder and file icons and not overlaying icon for folders.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved build-ready Customization to include into build custom MagentaCLOUD customisation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants