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

Contributing information should discuss contribution requirements for APG example code #88

Closed
karlgroves opened this issue Sep 8, 2015 · 5 comments

Comments

@karlgroves
Copy link

This repository's README document discloses contribution guidelines for the ARIA spec. The wiki has a table of Aria Authoring Practices and their status. Neither documents discuss details on requirements for submitting examples. Documenting the requirements (if any) will probably help in getting examples and ensure their quality.

@ZoeBijl ZoeBijl added the APG label Jan 12, 2016
@ZoeBijl
Copy link

ZoeBijl commented Jan 12, 2016

Can you elaborate? There are plans for some code standards:

APG itself: #131
APG examples: #95

Your issue predates both, but is this what you're hinting at? Or should we write a full fledged tutorial on how to contribute?

I'm open to both, and actually think the code examples should be written in a tutorial format (but that is a separate issue entirely).

@karlgroves
Copy link
Author

While coding standards are a good start, I'm looking for something else.

For instance, here's the kind of thing I'd propose if this was my own project:

  1. All examples must be a standalone demonstration of a specific concept. This includes all HTML, CSS, and JS without any external dependencies at all.
  2. All examples must come with a 'feature' file written in Gherkin syntax that descripts the specific goal of the example and specific user interactions necessary to use the example. Also, where appropriate, the roles, states, and properties that should be exposed to AT are described.
  3. All HTML and CSS should be valid.

With some additional effort I'm sure I could make this list even more oppressive, but the general idea is just to codify exactly what you expect so that contributors have some clarity on what they need to do.

@ZoeBijl
Copy link

ZoeBijl commented Jan 12, 2016

Good points, some comments:

All examples must be a standalone demonstration … without any external dependencies at all.

Do you mean that we should have two documents for each example (one for the actual code and one to document the example)? Or do you dislike all examples sharing a common CSS?

All examples must come with a 'feature' file written in Gherkin syntax

I've seen some talks on Cucumber and automated testing, but don't have any experience with it (this might be a good time to start). I'll look into it and will ask if others have experience and if they can help. Of course, writing the feature file seems easy enough.

All HTML and CSS should be valid.

Couldn't agree more. I have not validated the examples. Have filed an issue: #174

@karlgroves
Copy link
Author

Do you mean that we should have two documents for each example (one for the actual code and one to document the example)? Or do you dislike all examples sharing a common CSS?

I'd be on the fence about having common assets for the repo itself. I can see the possibility of providing a common set of files for convenience.

I guess it depends on the goal. For me, I'd still favor having a completely standalone example that could survive on its own.

[re: Cucumber] Of course, writing the feature file seems easy enough.

I wasn't necessarily proposing the use of Cucumber itself, but Gherkin syntax is a really excellent way to communicate requirements in a very clear and concise way that can be understood by everyone involved.

@ZoeBijl
Copy link

ZoeBijl commented Apr 29, 2016

This issue was moved to w3c/aria-practices#11

@ZoeBijl ZoeBijl closed this as completed Apr 29, 2016
pkra pushed a commit to pkra/aria that referenced this issue May 20, 2024
Added HTML-AAM reference to step 2E.
pkra pushed a commit that referenced this issue May 20, 2024
pkra pushed a commit that referenced this issue May 20, 2024
AX API: Complete mapping for 'time' element (Github issue #88)
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

No branches or pull requests

2 participants