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

Autocomplete and Date Picker Combobox Examples: Update to conform with latest APG coding guidelines #1623

Merged
merged 7 commits into from
Dec 1, 2020

Conversation

jongund
Copy link
Contributor

@jongund jongund commented Nov 18, 2020

Since this is an important example in the APG, I thought it would be important to update the code to the new coding practices to make it easier to read.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 18, 2020

Regression test coverage:

Examples without any regression tests:

  • dialog-modal/alertdialog.html

Examples missing some regression tests:

  • dialog-modal/datepicker-dialog.html:
    • textbox-aria-describedby
  • menu-button/menu-button-actions-active-descendant.html:
    • menu-up-arrow
    • menu-down-arrow
    • menu-character
  • toolbar/toolbar.html:
    • toolbar-tab
    • toolbar-right-arrow
    • toolbar-left-arrow
    • toolbar-home
    • toolbar-end
    • toolbar-toggle-esc
    • toolbar-toggle-enter-or-space
    • toolbar-radio-enter-or-space
    • toolbar-radio-down-arrow
    • toolbar-radio-up-arrow
    • toolbar-button-enter-or-space
    • toolbar-menubutton-enter-or-space-or-down-or-up
    • toolbar-menu-enter-or-space
    • toolbar-menu-down-arrow
    • toolbar-menu-up-arrow
    • toolbar-menu-escape
    • toolbar-spinbutton-down-arrow
    • toolbar-spinbutton-up-arrow
    • toolbar-spinbutton-page-down
    • toolbar-spinbutton-page-up
    • toolbar-checkbox-space
    • toolbar-link-enter-or-space
    • toolbar-spinbutton-role

Example pages with Keyboard or Attribute table rows that do not have data-test-ids:

  • dialog-modal/alertdialog.html
    • "Keyboard Support" table(s):
      • Tab
      • Shift + Tab
      • Escape
      • Command + S
      • Control + S
    • "Attributes" table(s):
      • alertdialog
      • aria-labelledby=IDREF
      • aria-describedby=IDREF
      • aria-modal=true
      • alert

SUMMARY:

55 example pages found.
1 example pages have no regression tests.
3 example pages are missing approximately 27 out of approximately 780 tests.

ERROR - missing tests:

Please write missing tests for this report to pass.

@spectranaut
Copy link
Contributor

spectranaut commented Nov 18, 2020

I looks like the merge conflicts are only related to the result of the linter running on the code. So you can resolve the code conflicts by checking out your copy of the file during the merge:

git merge master
// A conflict will be revealled
git checkout --ours ./examples/combobox/js/combobox-datepicker.js
git merge --continue

I'm not sure why we are seeing the error in github actions above, but I'm looking into it!

Copy link
Contributor

@spectranaut spectranaut left a comment

Choose a reason for hiding this comment

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

These changes look good and the example works as expected -- the merge conflicts just need to be addressed.

I can run npm regression-report locally on this branch and it succeeded, so I don't think the failure in the github checks comment here is related to that.

@mcking65 mcking65 changed the title Updated combox JS file code to use new coding practices Autocomplete and Date Picker Combobox Examples: Update to conform with latest APG coding guidelines Dec 1, 2020
@mcking65 mcking65 added enhancement Any addition or improvement that doesn't fix a code bug or prose inaccuracy Example Page Related to a page containing an example implementation of a pattern labels Dec 1, 2020
@mcking65 mcking65 added this to the 1.2 Release 1 milestone Dec 1, 2020
@mcking65
Copy link
Contributor

mcking65 commented Dec 1, 2020

Thank you @jongund

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Any addition or improvement that doesn't fix a code bug or prose inaccuracy Example Page Related to a page containing an example implementation of a pattern
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants