SITES-13120 - Image Smart Crop support for remote assets in various Sites core components

indra2gurjar commented 1 year ago

Q	A
Fixed Issues?	SITES-13120
Patch: Bug Fix?
Minor: New Feature?	SITES-13120
Major: Breaking Change?	No
Tests Added + Pass?	Yes
Documentation Provided	Yes (code comments and or markdown)
Any Dependency Changes?	No
License	Apache License, Version 2.0

Adding smartcrop functionility to NextGen dynamic media images. Added a new smartcrop button which will open only when a NGDM image is selected. On clicking the button, a dialog will open with UI for taking user input for smartcrop aspect ratios and image preview showing the smartcrop.

https://github.com/adobe/aem-core-wcm-components/assets/10485532/b810dc9d-f041-4340-8fe5-2b9cde91fa31

codecov[bot] commented 1 year ago

Codecov Report

Merging #2524 (ab600c3) into main (ce27843) will increase coverage by 0.03%. The diff coverage is 90.10%.

:exclamation: Current head ab600c3 differs from pull request most recent head 527fa4b. Consider uploading reports for the commit 527fa4b to get more accurate results

@@             Coverage Diff              @@
##               main    #2524      +/-   ##
============================================
+ Coverage     87.27%   87.31%   +0.03%     
- Complexity     2623     2645      +22     
============================================
  Files           229      232       +3     
  Lines          6986     7063      +77     
  Branches       1060     1071      +11     
============================================
+ Hits           6097     6167      +70     
- Misses          358      359       +1     
- Partials        531      537       +6

Impacted Files	Coverage Δ
...s/internal/models/v3/NextGenDMImageURIBuilder.java	`87.93% <87.93%> (ø)`
...nts/internal/models/v3/NextGenDMThumbnailImpl.java	`91.30% <91.30%> (ø)`
.../core/components/internal/models/v3/ImageImpl.java	`95.09% <100.00%> (+1.57%)`	:arrow_up:
...omponents/models/nextgendm/NextGenDMThumbnail.java	`100.00% <100.00%> (ø)`