Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
API changes to make FLEDGE understand ad sizes #417
API changes to make FLEDGE understand ad sizes #417
Changes from 1 commit
1a2368f
ff42d5b
d63d8e1
8851afc
d662fb7
038f2ca
7791d78
397e544
19c08f5
bbc82b5
0251bbc
3ef5be6
fc9653f
01ef829
a0a310d
1ddddee
d642777
f25854c
d119ceb
5380368
d102fbc
47d50e6
568873a
0a3db48
adf4cbc
cb668e8
733a821
3ec71ac
cfbcb70
7c6aa0e
85c2f60
1e07855
c1e00cf
2d44dd9
7625240
9ecc5f6
4efd2e8
064841f
8199508
81a8bf9
7b856f7
0cc20cb
21795b9
ed3b830
cd73790
06ace19
ca80dd8
910a9f2
3170266
b65b04a
4d4364b
6e6ec85
975e965
c3fad7a
cce7780
3e34e0d
a3662d5
d69f233
2d3f228
a57adbe
7148d74
f8cd3f6
50893b4
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the "inner frame size" here referring to the same thing as:
Bidders inside the auction may pick a different content size for the ad, and that resulting size will be visually scaled to fit inside the element's container size.
in the above section on requestSize?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, the inner frame size is the size seen by the ad creative, which is determined by the bidder. The container size is the size seen by the embedder, determined by the embedder (no size information is returned from the auction API call).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Back to the subject discussed on the Fledge call - https://github.com/WICG/turtledove/blob/main/meetings/2022-11-09-FLEDGE-call-minutes.md and in the github issue - #312 (comment):
to be able to implement carousel (or other type of animation in creative that present single product in two ways) we should define each product twice - which effectively - decrease this constraint to 10.
Could you consider two options:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We'll discuss this and get back to you. From my perspective, the latter option sounds pretty unobjectionable, but I wasn't involved in picking the original max number of components so there might be something I'm missing.