element-ios icon indicating copy to clipboard operation
element-ios copied to clipboard

Add NSUserActivity to rooms to appear in spotlight search

Open kloenk opened this issue 4 years ago • 3 comments

This also allows shortcuts to open the app directly opening a room.

Pull Request Checklist

  • [x] I read the contributing guide
  • [ ] UI change has been tested on both light and dark themes, in portrait and landscape orientations and on iPhone and iPad simulators
  • [x] Pull request is based on the develop branch
  • [x] Pull request contains a changelog file in ./changelog.d
  • [ ] Pull request includes screenshots or videos of UI changes
  • [x] Pull request includes a sign off

kloenk avatar Sep 18 '21 14:09 kloenk

For this initial implementation, would you mind removing the CSSearchableItemAttributeSet and CSSearchableItem parts and relying on the userActivity.isEligibleForSearch = true for spotlight support? I believe the only visible change this would have is the lack of a subtitle - which may be a positive thing as it could be confusing in the Shortcuts app.

It would be interesting to have spotlight functionality in its own service eventually, so that updates to the index can happen automatically. Then the UserActivityService could request the appropriate attributes to attach to the activity from that service.

pixlwave avatar Oct 27 '21 09:10 pixlwave

Let's anchor 3086234a479bfcd84b6106eb1a5831f6f4bbe337 here, so it's easier to find how to add it back later on

kloenk avatar Oct 31 '21 17:10 kloenk

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Sep 25 '24 11:09 CLAassistant