Skip to content

Fix safari issues and move scrollable to own container#7029

Merged
bramkragten merged 4 commits intodevfrom
media-browser-scroll-header
Sep 30, 2020
Merged

Fix safari issues and move scrollable to own container#7029
bramkragten merged 4 commits intodevfrom
media-browser-scroll-header

Conversation

@bramkragten
Copy link
Member

@bramkragten bramkragten commented Sep 16, 2020

Breaking change

Proposed change

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (thank you!)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Example configuration

Additional information

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

@bramkragten bramkragten marked this pull request as draft September 16, 2020 20:19
@zsarnett zsarnett self-requested a review September 17, 2020 00:58
@bramkragten bramkragten marked this pull request as ready for review September 29, 2020 20:18
Comment on lines +451 to +461
const header = this._header;
const content = this._content;
Copy link
Contributor

Choose a reason for hiding this comment

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

why?

Copy link
Member Author

Choose a reason for hiding this comment

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

Saves a few media queries.

Copy link
Contributor

@zsarnett zsarnett left a comment

Choose a reason for hiding this comment

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

I tested it on a few devices. Seems pretty good. 👍

@bramkragten bramkragten force-pushed the media-browser-scroll-header branch from 275b2c0 to 8af69b5 Compare September 30, 2020 08:45
@bramkragten bramkragten merged commit 4f2bad0 into dev Sep 30, 2020
@bramkragten bramkragten deleted the media-browser-scroll-header branch September 30, 2020 09:05
@bramkragten bramkragten mentioned this pull request Sep 30, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Jul 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Media Browser dialog header disappears after scrolling in Safari

3 participants