-
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
Colorbar overlay module with more advanced design #430
Comments
1 similar comment
can I work on this @jywarren ? |
That would be super - we'd love your help! |
I've added a new issue for setting the background during a crop operation: #467 |
So, this sequence is getting closer! https://sequencer.publiclab.org/examples/#steps=crop{x:0|y:0|w:611|h:479},gradient{},colormap{colormap:stretched},crop{x:0|y:0|w:611|h:60},resize{resize:25%25},overlay{x:5|y:5|offset:-4} But I still can't get it to appear in the newly added padding below the image that results from the crop. But closer! |
@tech4GT I would like to work on this issue! |
I am thinking to implement drawing rectangles module first, so that we can use it to display the colorbar with an outline! |
@jywarren I made something like this Is this what we require? |
After #285, we now can generate and overlay a colobar, and it looks great:
I'm thinking what the next step could be; we could potentially work on text overlay (currently in #165 but will be broken out) to show labels
and also display the colorbar with an outline (we also don't have "drawing rectangles" as a module... but we could! break this out too!)
The goal might be something like this:
We could also have the colorbar not be overlaid on the image, but shown below it. This could be done with the crop module maybe cropping at larger than the current canvas (although I'm not sure this will work -- would it leave white space?) https://github.com/publiclab/image-sequencer/blob/88364decf6a6111a0be31fd98010576d5c6f918e/src/modules/Crop/info.json
This could be broken into 3 different issues to solve.
The text was updated successfully, but these errors were encountered: