-
Notifications
You must be signed in to change notification settings - Fork 122
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
Campfire serve and open-browser #638
Conversation
5ab9cc7
to
2a5fb5c
Compare
36ec294
to
5ed3497
Compare
155b51d
to
2c59248
Compare
2c59248
to
be1cefa
Compare
@@ -323,7 +323,7 @@ fn material_filter_matches(filter: &MaterialFilter, mat: &PbrMaterialDesc) -> bo | |||
// } | |||
// } | |||
|
|||
pub const MODEL_EXTENSIONS: &[&str] = &["glb", "fbx", "obj", "x"]; | |||
pub const MODEL_EXTENSIONS: &[&str] = &["glb", "gltf", "fbx", "obj", "x"]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lol nice, I guess this fixes #644 as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, did it while pairing with @chaosprint and it just came up and we did it on the go while discussing how my edit-build-test workflow loop looks 😆
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, hope it all works!
// return false; | ||
// } | ||
|
||
match fname.to_str() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looking at this - maybe, as a future extension, it should respect .gitignore
? definitely not necessary now, just figured I'd plant the idea
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, figured as such as well.
Oh .DS_STORE
how I've missed you
Accidentally fixes #644 |
Improves the web workflow through campfire