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

Issue with "FileSystemHandle": (short summary here please) #2794

Closed
bboyle opened this issue Mar 2, 2021 · 2 comments · Fixed by #3168
Closed

Issue with "FileSystemHandle": (short summary here please) #2794

bboyle opened this issue Mar 2, 2021 · 2 comments · Fixed by #3168
Assignees
Labels
Content:WebAPI Web API docs effort: medium This task is a medium effort. p2 Minor issue with low priority, can be fixed later.

Comments

@bboyle
Copy link

bboyle commented Mar 2, 2021

MDN URL: https://developer.mozilla.org/en-US/docs/Web/API/FileSystemHandle

What information was incorrect, unhelpful, or incomplete?

I'm confused about FileSystemFileHandle vs FileSystemFileEntry on this page.

Specific section or headline?

  • top introduction
  • Interfaces based on FileSystemHandle

What did you expect to see?

The API for FileSystemHandle, FileSystemFileHandle and FileSystemDirectoryHandle
I'm not sure if entry interfaces belong here or not. I don't think FileSystemFileEntry inherits from this interface?

Did you test this? If so, how?

I am working with the API today. Chrome tells me that FileSystemFileHandle.file is not a function.

MDN Content page report details
@chrisdavidmills chrisdavidmills added the Content:WebAPI Web API docs label Mar 2, 2021
@wbamberg wbamberg self-assigned this Mar 7, 2021
@Rumyra
Copy link
Collaborator

Rumyra commented Mar 15, 2021

Suffice to say this opening paragraph is confusing - fileSystemHandle provide a handle to a fileSystemEntry. fileSystemFileHandle handles fileSystemFileEntry and fileSystemDirectoryHandle -> fileSystemDirectoryEntry

@wbamberg I wrote this originally so if you have any questions just shout 👍

@Rumyra Rumyra added effort: medium This task is a medium effort. p2 Minor issue with low priority, can be fixed later. labels Mar 15, 2021
@bboyle
Copy link
Author

bboyle commented Mar 17, 2021

thanks for making MDN great! :)

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Content:WebAPI Web API docs effort: medium This task is a medium effort. p2 Minor issue with low priority, can be fixed later.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants