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.

Sign up for GitHub

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

Jump to bottom

Add /layoutImage to /FloorPlan (in Pending)#2690

Closed
danbri opened this issue Aug 18, 2020 · 1 comment
Closed

Add /layoutImage to /FloorPlan (in Pending) #2690

danbri opened this issue Aug 18, 2020 · 1 comment
Assignees
@danbri

Comments

@danbri
Copy link
Contributor

danbri commented Aug 18, 2020

Context: this is a proposal from Google based on our review of the draft (i.e. Pending) definitions for /FloorPlan and nearby terms.

This proposal adds a /layoutImage to /FloorPlan as specified earlier in #2373

Noting that FloorPlan is place-related but an /Intangible rather than a /Place subtype (and therefore not currently supporting /photo) we propose adding a dedicated property more specific than /image to capture FloorPlan-specific imagery. This is a little unusual in that we don't expect schema.org needs properties for all the infinite range of things that can be depicted online; however the type FloorPlan itself is inherently image-oriented and it seems valuable to distinguish floorplan imagery from other images of rooms and apartments.

  • /layoutImage "A schematic image showing the floorplan layout."
  • domainIncludes: FloorPlan; rangeIncludes: ImageObject, url
  • super-property: /image
@danbri danbri self-assigned this Aug 18, 2020
@danbri danbri mentioned this issue Aug 18, 2020
Closed
@danbri
Copy link
Contributor Author

danbri commented Aug 18, 2020

I should have mentioned this comes partly as usability feedback, in that the type FloorPlan almost sounds like an image type, and yet we only have vague language ("The visual/spatial aspects of a floor plan (i.e. room layout, see wikipedia) can be indicated using image.") on how to actually find the plan imagery. This would be amended to reference /layoutImage instead of /image.

@danbri danbri closed this as completed in 681e08a Aug 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@danbri danbri

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests

1 participant
@danbri