Extend cpagefield image type - #4071
Conversation
|
Code Climate has analyzed commit c11b345 and detected 2 issues on this pull request. Here's the issue category breakdown:
The test coverage on the diff in this pull request is 60.0% (80% is the threshold). This pull request will bring the total coverage in the repository to 5.4% (0.0% change). View more on Code Climate. |
|
@veskoto And your popup should be solved in the template by calling image field 2x - one with raw parameter and one without it. The rest is HTML markup that can be different for another theme. |
|
Thank you @veskoto . I'll look into this.. the PR is not exactly how I think it should happen, but inside thumbUrl(). |
9119ebf to
2a0ee3f
Compare
241b734 to
4f50dac
Compare
$valueis null or empty{CPAGEFIELD: name=XXXX&type=full}return full path to image. This ignore thumb params.{CPAGEFIELD: name=XXXX&type=popup}return a wrapped image with link to full image path.