Skip to content

[FEATURE REQUEST] Review avoidScreenshotsIfNeeded usage #4996

Description

@jesmrec
fun Dialog.avoidScreenshotsIfNeeded() {
    if (!BuildConfig.DEBUG && context.resources?.getBoolean(R.bool.allow_screenshots) == false) {
        window?.addFlags(WindowManager.LayoutParams.FLAG_SECURE)
    }

such method is used in several dialogs across the app. Some of them (like the remove share or remove member from space) do not seem to need to avoid screenshots

All the uses of the method should be reviewed to keep those that are really useful, and remove the ones that don't need avoidScreenshotsIfNeeded

TASKS

  • Research (if needed)
  • Create branch feature/feature_name
  • Development tasks
    • Implement whatever
    • ...
    • Implement unit tests (if needed)
  • Code review and apply changes requested
  • Design test plan
  • QA
  • Merge branch feature/feature_name into master

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions