-
Notifications
You must be signed in to change notification settings - Fork 15
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
Clarify inclusion #185
Clarify inclusion #185
Conversation
This should probably be looked at from the lens of "does this satisfy the original issue #169?" |
@cwilso I don't think it does, but I wanted to not lose the improvements you proposed in your PR even if we close that issue with no change, or decide to defer it. |
@fantasai @cwilso What about economic situation, and age, including older and younger people? I found it on the WAI: https://www.w3.org/WAI/fundamentals/accessibility-usability-inclusion/.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
some of this is an improvement already, I think some needs more improvement
I think those are important different perspectives, and I think we should be clear about including them (right now they are in the catch-all "and more". The issue raised - #169 - was in part about what to say explicitly - do we want to be exhaustive, divide perspectives along different axes, make the document short, ...? I'm not sure there is a right answer. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe the changes being suggested here touch on specific approaches being discussed in #169 and don't have consensus there yet. Some of these look like improvements to me as well, however since there are already a bunch of opinions as to how to solve #169, I'd rather discuss those in that issue's comments directly instead of spread out across multiple PRs attempting to address the issue.
At this point, given the variety of opinions and approaches being expressed, I'd like to hold off on any PRs for #169 until we have found a way to reach consensus in our approaches to addressing it, in the comments in #169.
Thus defer (or even close) this PR while capturing the salient (or implicit) points/perspectives in this PR as deliberate changes with reasons in a comment in #169.
deferring (labeled as such) for now with editor+chair consensus, rather than closing, in the hopes that relevant points of discussion here are copied/moved to #169 by the PR author, who is then encouraged to close this PR without change. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As requested, I have taken another closer look at this PR and find that despite the attempts at readability improvement, it is actually a big step backwards in one particular way.
The issue is titled: "Clarify inclusion"
Whereas this PR ironically actually REMOVES both instances of "inclusion" from this text.
(Emphasis added)
That's an unacceptable substantive regression so I'm a strong -1 on this PR.
I can live with "defer", but as noted, I'd prefer (PROPOSE) to close without changes, and request that any proposed improvements in this text be instead added as a comment to issue #169 for others to discuss and seek a broader consensus than 1-2 people.
Note that neither this PR nor the issue it is purporting to help advance has either initially or subsequently been labeled a "blocker" or "needed for Note" (or suggested as such in comments therein), so in good faith I expect that it will NOT be used as a reason to block the CFC to publish an updated Vision Note.
I don't see the changes as editorial improvements. I support Tantek's proposal to close without changes and we can maybe revist after note publication if there is clarifying work in issue #169 |
I made one suggestion above to re-introduce "inclusion". Without that change, I have to agree with Tantek and Max that this is a regression in readability. With that, I'm comfortable with it being a slight improvement - as it separates the inclusion and diversity thoughts better - though it does not address the issue that started this. :) |
This edit pulls some of the improvements of PR w3c#171 without dividing into two separate points with independent lists of diversity attributes.
@cwilso Updated. |
In discussion with @tantek and @mgendler about this, trying to synthesize all the bits of feedback on the issue but particularly concerned about the redefinition of the term "diversity", we had an idea I'm going to try out. I don't want to close this PR, and I appreciate the work you've done on it, but I want to take another stab before we call this approach the right one. |
This PR would be obviated by #231 |
#231 has been merged, so this PR is no longer applicable. @fantasai, please take a look at how inclusion is covered in https://w3c.github.io/AB-public/Vision#diversity (and the preceding principle on stakeholders) and make sure there isn't additional concerns you have. |
https://github.com/w3c/AB-public/pull/231/files works for me |
This edit pulls some of the improvements of PR #171 without dividing into two separate points with independent lists of diversity attributes.
Preview | Diff