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

App Picker #2499

Merged
merged 13 commits into from
Jan 17, 2025
Merged

App Picker #2499

merged 13 commits into from
Jan 17, 2025

Conversation

zeitschlag
Copy link
Collaborator

@zeitschlag zeitschlag commented Jan 9, 2025

Adds an App Picker to download apps from webxdc.org/apps. It works like this:

  1. User taps on the new "Apps"-menu-entry after opening the attachmenet-menu in a chat (the previous webxdc-selector got basically replaced)
  2. The Picker opens, lets the user browse the website and pick an app
  3. That app gets downloaded and staged as a draft in that very chat
It looks like this

2450
Simulator Screenshot - iPad Pro 11-inch (M4) - 2025-01-17 at 10 46 17


  • Only one entry "apps" in menu -> Remove the existing app-picker for now
  • Consider keyboard It's a mess due to webview (or I'm incompetent), so I'll leave this for now as it is.
  • Loading Indicator

Closes #2450 eventually

@zeitschlag zeitschlag self-assigned this Jan 9, 2025
@r10s
Copy link
Member

r10s commented Jan 10, 2025

pretty cool already!

@r10s r10s added the enhancement actually in development, user visible enhancement label Jan 12, 2025
@zeitschlag zeitschlag force-pushed the 2450-app-picker branch 3 times, most recently from f01553c to a51dc8a Compare January 17, 2025 09:44
@zeitschlag zeitschlag requested a review from r10s January 17, 2025 09:56
@zeitschlag zeitschlag marked this pull request as ready for review January 17, 2025 09:56
@zeitschlag zeitschlag changed the title [WIP] App Picker App Picker Jan 17, 2025
Copy link
Member

@r10s r10s left a comment

Choose a reason for hiding this comment

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

again, very nice! 👍

deltachat-ios/Chat/AppPickerViewController.swift Outdated Show resolved Hide resolved
deltachat-ios/Chat/AppPickerViewController.swift Outdated Show resolved Hide resolved
@zeitschlag zeitschlag merged commit 88599e6 into main Jan 17, 2025
2 checks passed
@zeitschlag zeitschlag deleted the 2450-app-picker branch January 17, 2025 14:38
let appPicker = AppPickerViewController()
appPicker.delegate = self
let navigationController = UINavigationController(rootViewController: appPicker)
navigationController.isModalInPresentation = true
Copy link
Collaborator

Choose a reason for hiding this comment

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

I really dislike that you can't dismiss this interactively. It makes sense during download but not really any other time.

Copy link
Member

Choose a reason for hiding this comment

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

you mean "swipe down to close"? idk, "files" and "gallery" have the same behavior, therefore i thought not too much about it. if it is an easy change, however, of couse, i would not be against having the same behavior as our contact or profile-switcher :)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Hmmm, that is actually an unintended side effect of the first responder fixes I did and did not notice :/

Copy link
Member

Choose a reason for hiding this comment

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

ah! :)

Copy link
Collaborator

Choose a reason for hiding this comment

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

I attempted to fix that today in many different ways but I could not figure out how to do it without swizzling. I have a PR in draft that is working with swizzled methods. #2536

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement actually in development, user visible enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

App Picker
3 participants