Skip to content
This repository has been archived by the owner on May 14, 2022. It is now read-only.

预览功能还需改进 #41

Open
Micraow opened this issue Jul 23, 2021 · 2 comments
Open

预览功能还需改进 #41

Micraow opened this issue Jul 23, 2021 · 2 comments
Labels
enhancement New feature or request

Comments

@Micraow
Copy link

Micraow commented Jul 23, 2021

我阅读了代码,发现好像这个程序是能够预览office 文件的,但是实际情况下却显示无法预览。是不是判断文件类型的地方写错了?(我不是做javascript,所以也没怎么了解,直觉而已)

另外,这个项目用来预览视频和音乐的播放器应该是dplayer吧,这个播放器是支持很多格式的,但是这个项目里仍然没有将他们的文件类型正确判断,导致无法进行预览。

举个例子: rmvb
dplayer是支持这个格式的,但是在你的程序中仍然无法预览。

@Micraow
Copy link
Author

Micraow commented Jul 23, 2021

补充: pdf也有此问题。

@Micraow
Copy link
Author

Micraow commented Jul 23, 2021

出现此问题的原因还有可能是: Microsoft Graph API传回来的MIME不总是对的,我觉得应该通过文件名后缀判断。(都能列表,文件名后缀的提取应该不成问题)

我看spencerwoo的做得挺不错,可以借鉴一下。

onedrive-cf-index

文件预览的代码

https://github.com/spencerwooo/onedrive-cf-index/blob/main/src/fileView.js

@ukuq ukuq added the enhancement New feature or request label Jul 24, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants