Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@savar
Copy link
Contributor

@savar savar commented Mar 14, 2017

two new (optional) parameters added to the watermark filter to resize
the watermark relatively to the image it should be placed on

see the provided doc file to understand how it works

@savar savar force-pushed the watermark_with_resize branch from 4538291 to 18fbd43 Compare March 14, 2017 20:53
@savar
Copy link
Contributor Author

savar commented Mar 15, 2017

should fix #874

@savar savar force-pushed the watermark_with_resize branch from 18fbd43 to fddcf45 Compare March 15, 2017 14:23
@r0bdiabl0
Copy link

This looks great.

@savar
Copy link
Contributor Author

savar commented Apr 28, 2017

anything which needs to be done to get this into thumbor?

@savar savar force-pushed the watermark_with_resize branch from fddcf45 to 554d24c Compare August 29, 2017 19:47
@savar
Copy link
Contributor Author

savar commented Aug 29, 2017

@masom ? 😄

@coveralls
Copy link

coveralls commented Aug 29, 2017

Coverage Status

Changes Unknown when pulling e4dfdb6 on Savar:watermark_with_resize into ** on thumbor:master**.

savar added 3 commits August 30, 2017 14:23
two new (optional) parameters added to the watermark filter to resize
the watermark relatively to the image it should be placed on

see the provided doc file to understand how it works
@savar savar force-pushed the watermark_with_resize branch from e4dfdb6 to bb8d4dc Compare August 30, 2017 12:25
@savar
Copy link
Contributor Author

savar commented Aug 30, 2017

@masom how about that one?

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.007%) to 84.534% when pulling bb8d4dc on Savar:watermark_with_resize into 1fc40b5 on thumbor:master.

@masom masom merged commit 746e550 into thumbor:master Aug 30, 2017
@savar savar deleted the watermark_with_resize branch August 30, 2017 13:29
@LorisLunedei
Copy link

That's great! When will it be released?

@Kmaschta Kmaschta mentioned this pull request Jan 5, 2018
9 tasks
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.

5 participants