Skip to content

ci: update remote browsers we test against #19961

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

Merged
merged 3 commits into from
Jul 21, 2020

Conversation

devversion
Copy link
Member

@devversion devversion commented Jul 13, 2020

  • See individual commits

@googlebot googlebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Jul 13, 2020
Updates our remote browsers so that we test against
the latest Egde version, the latest iOS and macOS Safari
versions.
@devversion devversion force-pushed the build/update-remote-browsers branch from e000b3e to 9a56b0d Compare July 13, 2020 18:29
@devversion devversion added the in progress This issue is currently in progress label Jul 13, 2020
…ctly for videos, audio and object elements

The interactivity checker has been built a while ago when supported
browsers behaved differently for their `tabbable` state. We now
updated our browsers to more recent versions, and need to adjust
the interactivity checker so that it reflects the latest patterns
in supported browsers.

Here is a note summary of behavior noticed for `video` and `audio`
elements in various supported browsers:
https://hackmd.io/@devversion/rkxzcgsJD

Another change was for `object` elements. In previous BLINK / Webkit
browsers, such elements, or its children were never tabbable. This
is incorrect for the currently supported browsers as children inside
`object` elements can certainly be tabbable. On the other hand though,
object elements itself can be tabbable depending on configuration
we cannot reliably detect. For improved accessibility, we never
consider object elements itself as tabbable (similar to iframes)
* Updates the MDC-based slider tests to work in Safari v13.1
* Updates the MDC-based slider tests so that the measurements
are guaranteed to be based on a 100px wide slider (seems like this
is not always the case sometimes; resulting in flakiness).
@devversion devversion added P2 The issue is important to a large percentage of users, with a workaround merge: preserve commits When the PR is merged, a rebase and merge should be performed target: patch This PR is targeted for the next patch release and removed in progress This issue is currently in progress labels Jul 14, 2020
@devversion devversion marked this pull request as ready for review July 14, 2020 22:10
@devversion devversion requested review from jelbourn and a team as code owners July 14, 2020 22:10
@devversion
Copy link
Member Author

Note that there is a second PR for updating Firefox. That one is managed by Bazel so it's more complicated. I'm working on a version of Firefox we can control in: angular/angular#38029

Copy link
Member

@jelbourn jelbourn left a comment

Choose a reason for hiding this comment

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

LGTM

@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Aug 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker cla: yes PR author has agreed to Google's Contributor License Agreement merge: preserve commits When the PR is merged, a rebase and merge should be performed P2 The issue is important to a large percentage of users, with a workaround target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants