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
Recipe #0489 Multimedia Canvas #507
Recipe #0489 Multimedia Canvas #507
Changes from 6 commits
74c0491
ed1e3ce
50e776a
6f08ad0
4dce5d9
795330f
8b767f2
9050099
659d884
10f2d77
4344ffc
2fb9bdb
e860842
1f22f9e
c5e8284
00a5c73
58bf6a3
951af55
f517cb2
3685e81
5944ad6
53e17c6
5d3bc67
60f2d24
7e61c55
5564d79
221c611
9e946e9
fcdf7d7
b5795ed
55461cc
0750ded
6e66cbc
54ab4cc
fff3e89
09dd1f9
2e74e4f
b2f826f
9249c64
1603085
48dd5b5
bb4e212
4e5b468
6ebc41d
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.
There is no selector on the resource, so because it's painted to:
t=27,57
I've assumed that the selector/cropping ist=0,27
. Is that a correct assumption?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.
Things are in-flight enough that it's a correct assumption given the current state of things, but it may become an incorrect one as I continue to edit. Right now, everything has been for placement only, to see whether I could get it syntactically valid and perhaps even working in a viewer. Now that you've shown me Theseus (thank you very much!) I can do something with the manifest and resources that might approach an authentic implementation of the use case. As part of that, I'll tighten up the syntax and make sure the property values are appropriate.