Skip to content

Update README to clarify DirPlayer's functionality#80

Open
LeoKids wants to merge 3 commits intoigorlira:mainfrom
LeoKids:patch-1
Open

Update README to clarify DirPlayer's functionality#80
LeoKids wants to merge 3 commits intoigorlira:mainfrom
LeoKids:patch-1

Conversation

@LeoKids
Copy link

@LeoKids LeoKids commented Mar 18, 2026

Clarified the purpose of DirPlayer and added a note about Shockwave Flash games.

Clarified the purpose of DirPlayer and added a note about Shockwave Flash games.
@chameleonxxl
Copy link
Collaborator

I can understand that you might be angry about it that we can't provide updates faster, but you need to keep in mind that we're offering dirplayer for free and working on it in your spare time.

So please have some more patience.

@markhughes
Copy link
Contributor

Really cool you wanted to contribute, I see and hear what you're putting out here. But I think a clearer approach is to instead outline the file types supported.

For updating the README its probably something to discuss first than to make a PR for straight away - perhaps having a discussion in the discord is best first.

@LeoKids
Copy link
Author

LeoKids commented Mar 21, 2026

Resolved,may you review this again? @markhughes

@LeoKids
Copy link
Author

LeoKids commented Mar 21, 2026

I've fixed that,can you review again?

DirPlayer is a Shockwave Player emulator written in Rust that aims to make playing old browser games possible on modern browsers.
DirPlayer is a Shockwave Player emulator written in Rust that aims to make playing applications and games made using Director possible on modern browsers.

This cannot run Flash games. For those, you can use Ruffle.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is (or should be) no expectation that a Shockwave emulator would run Flash games. We are working on adding support for Flash content *embedded* in Shockwave movies using Ruffle, but playing top-level SWF files remains out of scope for this project, and as it is currently written, this line creates more questions than it answers. If you really feel like this should be highlighted, I agree with @markhughes's suggestion to outline supported file formats as it would be a much clearer way to indicate this.

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

Successfully merging this pull request may close these issues.

4 participants