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

eXIf Exchangeable Image File (Exif) Profile #28

Closed
svgeesus opened this issue Jun 29, 2021 · 6 comments
Closed

eXIf Exchangeable Image File (Exif) Profile #28

svgeesus opened this issue Jun 29, 2021 · 6 comments
Labels
documentation Improvements or additions to documentation
Milestone

Comments

@svgeesus
Copy link
Contributor

The eXIf Exchangeable Image File (Exif) Profile chunk should probably be added to the main PNG specification, rather than being off to the side in an extensions document.

@svgeesus svgeesus added the documentation Improvements or additions to documentation label Jun 29, 2021
@svgeesus
Copy link
Contributor Author

Raised by Phil Harvey phil@owl.phy.queensu.ca

Although the original eXIf specification didn't mandate a location for the eXIf chunk,
web developers are advising to ignore the eXIf if it comes after IDAT:

w3c/csswg-drafts#4929

As such, you may want to consider amending the specification to reflect this.
Minimally, it should be recommend to place eXIf before IDAT for web compatibility.
Or possibly requiring it to come before IDAT.

@svgeesus
Copy link
Contributor Author

Pull request #81

@svgeesus
Copy link
Contributor Author

As such, you may want to consider amending the specification to reflect this.
Minimally, it should be recommend to place eXIf before IDAT for web compatibility.
Or possibly requiring it to come before IDAT.

Note that the above PR just merges in the old text from the extensions document.

I agree that this restriction should be added, but plan to do that as a separate change.

@ProgramMax
Copy link
Collaborator

With pull request #81 merged, can this issue be closed?

Should the separate change for the restriction also have a separate issue?

@ProgramMax
Copy link
Collaborator

Ping

@svgeesus
Copy link
Contributor Author

Yes this is all in the spec now

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

No branches or pull requests

2 participants