Bug #4452
Courier IntentFilters claim all HTTP and HTTPS URLs
| Status: | Resolved | Start date: | 01/25/2015 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: | vanevery | % Done: | 0% | |
| Category: | - | Spent time: | - | |
| Target version: | - | |||
| Component: |
Description
When opening an URL, Courier is always proposed in the list of compatible apps, whatever the type of the resource the URL represents.
It's quite annoying since if I want to be able to open an URL with Courier later I cannot cost a standard web browser from the list to always open URL.
Is it possible to associate Courier only with feeds ?
Associated revisions
make feed Intent Filters targetted to only feeds
The original IntentFilters made Courier show up as an option for all http
and https links. But Courier cannot handle all http/s links. So for some
random webpage, the link is placed in the "Add Feed" field, but when you
click Add, it fails unless its an RSS feed.
This also makes clicking on a proper feed work better since Courier provides
the only matching pattern, when the pattern is more specific, and therefore
Courier is the option option, and the process skips the menu to chose which
app, and instead the feed goes straight into the Add Feed page.
closes #4452 https://dev.guardianproject.info/issues/4452
(cherry picked from commit 638c872931bd50466f7e2f5a1a3f4d0c987ca8a6)
History
#1 Updated by hans almost 3 years ago
- Subject changed from App is registered with all kind of URL to Courier IntentFilters claim all HTTP and HTTPS URLs
- Assignee set to vanevery
I agree, this is very annoying and often confusing. I almost have a pull request done to fix this.
#2 Updated by hans almost 3 years ago
@n8fr8 said:
BTW, with the Courier ability to load any link... i think we wanted you to be able to open any page, and then have Courier scrape/scour that page for RSS links like the ones above.
It would be great if Courier can do that, but now it is far from doing that, so the IntentFilters should match its capability to avoid confusing users with extra clutter and noise.
#3 Updated by vanevery almost 3 years ago
We have the proper set intent filters commented out with the liberal set commented in. Will revert to the proper, more restrictive set.
#4 Updated by Anonymous over 2 years ago
- Status changed from New to Resolved
Applied in changeset securereader|commit:9018f3ff79b0b8bb65b71574734434854f313366.