Skip to content
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

Tweak: Change expand image AI feature to use Clipdrop uncrop [ED-14685] #27359

Merged
merged 13 commits into from
May 20, 2024

Conversation

netanelavr
Copy link
Member

@netanelavr netanelavr commented May 17, 2024

PR Checklist

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

Summary

This PR can be summarized in the following changelog entry:

  • Update expand image AI feature to use Clipdrop uncrop API

Description

  • remove the prompt field from the expand image view as it's not needed
  • decrease the max scale to 1 to only allow expanding the image
  • add two callbacks from the OutPaintingContent to get the image size and position
  • pass position, size and image_url to the ai-api

image

https://github.com/elementor/elementor-ai-api/pull/750

Test instructions

This PR can be tested by following these steps:

Quality assurance

  • I have tested this code to the best of my abilities
  • I have added unittests to verify the code works as intended
  • Docs have been added / updated (for bug fixes / features)

Fixes #

modules/ai/module.php Show resolved Hide resolved
@netanelavr netanelavr merged commit 50be85d into elementor:main May 20, 2024
52 checks passed
netanelavr added a commit to netanelavr/elementor that referenced this pull request May 20, 2024
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.

None yet

3 participants