-
Notifications
You must be signed in to change notification settings - Fork 210
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
New shadow module added #816
Conversation
Thanks for opening this pull request! |
Nirav. I have a few suggestions.
|
After doing grunt build, the shadow module is visible on my local. I changed info.json and Module.js to create a new metamodule, on module load I get this error image-sequencer.js:72130 Uncaught TypeError: Cannot read property 'format' of undefined I think something is wrong with info.json , I am updating this PR, please have a look at the code @harshkhandeparkar |
@niravasher please mention @publiclab/is-reviewers for reviewing it. |
Please checkout as your travis-ci failed... |
I think you will have to improve upon the gradient module:
You can try implementing these features 😊. Thanks |
Gradients for issue #822 right @harshkhandeparkar ? |
@niravasher what I meant was that limitations of the gradient module will only create a shadow in a single direction and single intensity in this shadow module. So I felt like the gradient module can be improved upon by implementing the changes I suggested. Thanks. |
Ok, looking into it @harshkhandeparkar |
I wonder if we tried the sequence i suggested before, like:
So
|
working on issue #806
Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!
npm test
@publiclab/reviewers
and@publiclab/is-reviewers
for help, in a comment belowIf tests do fail, click on the red
X
to learn why by reading the logs.Please be sure you've reviewed our contribution guidelines at https://publiclab.org/contributing-to-public-lab-software
Thanks!