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

Support sticky ToC on Safari #5113

Merged
merged 1 commit into from Nov 20, 2018

Conversation

Projects
None yet
3 participants
@rik
Copy link
Contributor

rik commented Nov 16, 2018

Safari only supports the prefixed version at the moment.

Support sticky ToC on Safari
Safari only supports the prefixed version at the moment.

@jwhitlock jwhitlock requested a review from schalkneethling Nov 16, 2018

@jwhitlock

This comment has been minimized.

Copy link
Member

jwhitlock commented Nov 16, 2018

Thanks @rik! Our guidelines require a bug in Bugzilla for code changes - would you be willing to file?

Assigning review to @schalkneethling, feel free to bounce to someone else.

@schalkneethling schalkneethling self-assigned this Nov 16, 2018

@rik

This comment has been minimized.

Copy link
Contributor

rik commented Nov 16, 2018

@schalkneethling
Copy link
Collaborator

schalkneethling left a comment

r+ Thanks @rik

@schalkneethling schalkneethling merged commit 1e87e44 into mozilla:master Nov 20, 2018

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
security/snyk - package.json (mdn) No new, high severity issues
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment