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

[WIP] add forwards compatibility / graceful degradation principle. #468

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

hober
Copy link
Contributor

@hober hober commented Jan 24, 2024

Fixes #175.


Preview | Diff

@hober hober self-assigned this Jan 24, 2024
@LeaVerou
Copy link
Member

LeaVerou commented Jan 24, 2024

Do we actually need two separate principles here or can this be merged with #354 / #469 ?

At the very least, they should be consecutive in the principles doc.

@@ -294,6 +294,7 @@ Parts of the web platform evolve independently.
Issues that are present with a certain web technology now may be fixed in a subsequent iteration.
Duplicating these issues makes fixing them more difficult.
By adhering to this principle we can make sure overall platform quality improves over time.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vertical whitespace is useful. Why would you remove the space around the h2?

(A more serious problem here is the lack of content between the h2 and the first h3. That's very bad form.

@@ -490,6 +491,120 @@ See also:
* [[#do-not-expose-use-of-assistive-tech]]
* [[#secure-context]]

<h3 id=degrade-gracefully>Degrade Gracefully</h3>
<!-- was "Degrade Gracefully" in the HTML Design Principles -->

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TL:DR; summary please!

index.bs Outdated
@@ -490,6 +491,120 @@ See also:
* [[#do-not-expose-use-of-assistive-tech]]
* [[#secure-context]]

<h3 id=degrade-gracefully>Degrade Gracefully</h3>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
<h3 id=degrade-gracefully>Degrade Gracefully</h3>
<h3 id=degrade-gracefully>Degrade gracefully</h3>

Copy link
Contributor

@martinthomson martinthomson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure that this text, which is very verbose, gets at the heart of the principle.

The key design decisions have already been made, so at best this is really explaining those decisions.

For instance, it was decided that an unrecognized HTML element would behave in a certain way; or that unknown CSS rules could be ignored; or that you need to check that a JS thingo exists before using it.

There are also things that apply to the use of Web-things that include consideration for the capabilities of the client, but I think we're not entirely within the remit of the document (modulo what Lea is doing for developers - there might be something to some adding some guidance there). However, most of those things require that someone act to ensure wide compatibility. Like checking for capabilities before using them, or like using polyfills (which obviously require that the fundamentals are implemented). Or they could just look at User-Agent and ship three versions of their site as seems to be increasingly common. Either way, I'm not sure that this is the place to start a dissertation on the subject.

@@ -294,6 +294,7 @@ Parts of the web platform evolve independently.
Issues that are present with a certain web technology now may be fixed in a subsequent iteration.
Duplicating these issues makes fixing them more difficult.
By adhering to this principle we can make sure overall platform quality improves over time.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vertical whitespace is useful. Why would you remove the space around the h2?

(A more serious problem here is the lack of content between the h2 and the first h3. That's very bad form.

@hober
Copy link
Contributor Author

hober commented Jan 25, 2024

I'm not sure that this text, which is very verbose, gets at the heart of the principle.

Oh, indeed. This is just a cut-and-paste of the text from the old HTML design principle. I've not posted anything worth reviewing yet.

@torgo torgo added this to the 2024-03-04-week milestone Mar 3, 2024
@plinss plinss added the Status: In Progress We're working on it but ideas not fully formed yet. label Mar 5, 2024
@torgo torgo modified the milestones: 2024-04-01-week, 2024-06-03-week Jun 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: In Progress We're working on it but ideas not fully formed yet.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add principle on forwards compatibility / graceful degradation (HTML Design Principle 2.2)
6 participants