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

Editorial: Refine the definition of an installable web application #1164

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

benfrancis
Copy link
Member

@benfrancis benfrancis commented Feb 8, 2025

PR #1163 added a definition of installable web applications. After code review comments had been addressed the resulting text included the sentence:

Any website is an installable web application.

I don't think that the vast majority of web developers would agree with this statement, and to me it seems to directly conflict with the text that follows which defines an installed web application as one which has (or will have) a manifest applied to its top level browsing context. To me this clearly indicates that a website can not be an installable web application unless it at least has a web application manifest, if only an empty manifest to which defaults can be applied. A manifest also can not be processed without a manifest URL (since it is an essential input to the processing algorithm).

This PR is a suggestion to refine that text to address my review comments in #1163.

Specifically:

  • What differentiates an installable web application from other websites is that it links to a web application manifest
  • A manifest is applied when an installed web application is launched

This change:

  • Makes editorial changes (changes informative sections, or changes normative sections without changing behavior)

Commit message:

Refine the definition of an installable web application

Person merging, please make sure that commits are squashed with one of the following as a commit message prefix:

  • chore:
  • editorial:
  • BREAKING CHANGE:
  • And use none if it's a normative change

Preview | Diff

@benfrancis benfrancis requested a review from dmurph February 8, 2025 08:02
@benfrancis benfrancis force-pushed the installable-web-apps branch from d668920 to 2be82b8 Compare February 8, 2025 08:38
@benfrancis benfrancis changed the title editorial: Refine the definition of an installable web application Editorial: Refine the definition of an installable web application Feb 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant