Skip to content

Fix up #13670#13901

Merged
seanbudd merged 1 commit into
masterfrom
fixupTableCell
Jul 14, 2022
Merged

Fix up #13670#13901
seanbudd merged 1 commit into
masterfrom
fixupTableCell

Conversation

@seanbudd

Copy link
Copy Markdown
Member

Link to issue number:

Summary of the issue:

There was a small typo in #13670

Description of user facing changes

Description of development approach

Fixes _TableCell.collSpan usage to _TableCell.colSpan

Testing strategy:

Known issues with pull request:

Change log entries:

unreleased regression

Code Review Checklist:

  • Pull Request description:
    • description is up to date
    • change log entries
  • Testing:
    • Unit tests
    • System (end to end) tests
    • Manual testing
  • API is compatible with existing add-ons.
  • Documentation:
    • User Documentation
    • Developer / Technical Documentation
    • Context sensitive help for GUI changes
  • UX of all users considered:
    • Speech
    • Braille
    • Low Vision
    • Different web browsers
    • Localization in other languages / culture than English

@seanbudd seanbudd requested a review from a team as a code owner July 13, 2022 22:59
@seanbudd seanbudd requested a review from feerrenrut July 13, 2022 22:59
@seanbudd seanbudd merged commit 432579d into master Jul 14, 2022
@seanbudd seanbudd deleted the fixupTableCell branch July 14, 2022 03:14
@nvaccessAuto nvaccessAuto added this to the 2022.3 milestone Jul 14, 2022
seanbudd added a commit that referenced this pull request Jul 25, 2022
This reverts commit 432579d.
seanbudd added a commit that referenced this pull request Jul 27, 2022
Reverts: #13670, #13901
Fixes #13927
Re-introduces: #13469

Summary of the issue:
The fix for #13927 is complex, and will need weeks of testing on alpha.
This means that #13469 is blocked until #13670 can be implemented with #13927 fixed.

Description of user facing changes
Removes table say all commands from 2022.3

Description of development approach
Revert PRs in the right order, fix up the release blurb

Testing strategy:
#13670 has been confirmed as the commit which broke #13927 using a git bisect.
The build from this code has been tested with bookworm.
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.

3 participants