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.

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

Use id/type instead of @id/@type #590

Closed
azaroth42 opened this issue Oct 28, 2015 · 15 comments
Closed

Use id/type instead of @id/@type #590

azaroth42 opened this issue Oct 28, 2015 · 15 comments

Comments

@azaroth42
Copy link
Member

Schema.org, Social Web / ActivityStreams, Annotation WG will all adopt this.
Manu, danbri, tantek, etc all agree.

There will be a best practices document published from W3C that states this

@azaroth42 azaroth42 self-assigned this Oct 28, 2015
@azaroth42 azaroth42 added this to the Image 3.0 milestone Oct 28, 2015
@azaroth42
Copy link
Member Author

We should confirm that we're going to do this to avoid future discussion.
My vote: 👍

@mikeapp
Copy link
Member

mikeapp commented Sep 26, 2016

👍

1 similar comment
@jpstroop
Copy link
Member

👍

@tomcrane
Copy link
Contributor

👍 - Went down well in The Hague

@zimeon
Copy link
Member

zimeon commented Oct 23, 2016

👍 - I was a little concerned about the churn this causes in code using the JSON but speaking with developers @ The Hague they seemed to see the change as a net benefit

@azaroth42 azaroth42 removed the defer label Jul 11, 2017
@azaroth42
Copy link
Member Author

Discussion on 8/2/17: Agreement. Need to make it clear that P3 + I2 will run into both.

Could we add id and type as well as leaving in @id and @type in Image 2.2? Fewer implementations of actual processors for image -- leaflet and OSD can support both and no one will notice :)

@jpstroop
Copy link
Member

jpstroop commented Aug 2, 2017

See also: #1064

@azaroth42
Copy link
Member Author

Manifest could consistently use id as it just gets fed to the Image processor anyway. Support from @jronallo and @mejackreed for this, as implementers.

@azaroth42 azaroth42 modified the milestones: Image 3.0, Presentation 3.0 Sep 20, 2017
@azaroth42 azaroth42 changed the title Emerging best practice is to use id/type instead of @id/@type Use id/type instead of @id/@type Sep 28, 2017
@zimeon
Copy link
Member

zimeon commented Oct 12, 2017

👍 agreed at Toronto WG meeting

@azaroth42
Copy link
Member Author

Done for Presentation, removing the tag, but leaving open until all the other specs catch up.

@azaroth42 azaroth42 removed this from the Presentation 3.0 milestone Nov 3, 2017
@azaroth42 azaroth42 added this to the Image 3.0 milestone Nov 3, 2017
This was referenced Dec 20, 2017
@azaroth42
Copy link
Member Author

Tagging principles to document as a design pattern.

@azaroth42 azaroth42 removed their assignment Feb 9, 2018
@azaroth42 azaroth42 mentioned this issue Feb 20, 2018
@azaroth42 azaroth42 removed the image label Feb 20, 2018
@azaroth42 azaroth42 removed this from the Image 3.0 milestone Feb 20, 2018
@azaroth42
Copy link
Member Author

Propose that this will be closed by documenting the principle, that we will then stick to.
E.g. by #1510

@zimeon
Copy link
Member

zimeon commented Mar 22, 2018

👍 Agree that this is already addressed in the v3 specs, can close by documenting principle in #1510

@tomcrane
Copy link
Contributor

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants