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

Allow selction of default pdf reader in settings/hamburger menu #74

Closed
e-alfred opened this issue Jan 29, 2018 · 2 comments
Closed

Allow selction of default pdf reader in settings/hamburger menu #74

e-alfred opened this issue Jan 29, 2018 · 2 comments

Comments

@e-alfred
Copy link

Hello,

now that you support pdf files (which is quite awesome), it would be great to be able to select which viewer should be used (either Nextcloud built in or the Ebook reader app).

@e-alfred e-alfred changed the title Allow selction of default pdf read in settings/hamburger menu Allow selction of default pdf reader in settings/hamburger menu Jan 29, 2018
@Yetangitu
Copy link
Owner

Yetangitu commented Jan 29, 2018

See nextcloud/viewer#2393 and owncloud/core#30306 for a proposed solution to this problem, it should really be handled in Core instead of in individual apps as the latter would lead to app turf wars fighting to the last bit of which app gets to handle which mime type.

@Yetangitu
Copy link
Owner

In v1.2.0 I implemented a temporary personal preferences section for this purpose where you can define for which file types Reader should be the default viewer:

image

Once nextcloud/viewer#2393 and owncloud/core#30306 have been implemented this section will be removed.

v1.2.0 will be released Real Soon Now...

Yetangitu added a commit that referenced this issue Jan 31, 2018
change (#73), default viewer selection in personal preferences section (#74), fixed undefined $title in template (#72)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants