Skip to content

feat: sticky method, property, and event headers #336

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 6 commits into from
Mar 29, 2019

Conversation

knownasilya
Copy link
Contributor

sticky

I'm 95% satisfied with this, the other 5% is if the section is really small I'd prefer not to show a sticky, but the addon just doesn't support it yet. I submitted a feature request here kaliber5/ember-sticky-element#4

@locks locks temporarily deployed to ember-api-docs-staging-pr-336 October 20, 2017 14:16 Inactive
@locks locks temporarily deployed to ember-api-docs-staging-pr-336 October 20, 2017 14:35 Inactive
@sivakumar-kailasam
Copy link
Member

@knownasilya Will this be sticky on mobile as well?

@knownasilya
Copy link
Contributor Author

Yes

@locks locks temporarily deployed to ember-api-docs-staging-pr-336 October 20, 2017 19:06 Inactive
@locks locks temporarily deployed to ember-api-docs-staging-pr-336 October 22, 2017 06:02 Inactive
@simonihmig
Copy link

Fyi: the feature request has been implemented in kaliber5/ember-sticky-element#12 (though differently, as was discussed). @knownasilya please let me know in case it does not work for you as expected!

@locks locks temporarily deployed to ember-api-docs-staging-pr-336 December 7, 2017 19:23 Inactive
@locks locks temporarily deployed to ember-api-docs-review-t-pr-336 March 29, 2019 05:41 Inactive
@toddjordan
Copy link
Contributor

Thanks for this fix, I tried this out on a few browser types and seems to work, but I'd like to use the current version of this addon (0.2.1) as it removes jquery in newer versions, plus your fix request will be implemented. @knownasilya do you have any bandwidth to update?

@knownasilya
Copy link
Contributor Author

Sure, I'll have a look today.

@knownasilya
Copy link
Contributor Author

@toddjordan should be all set.

@toddjordan toddjordan merged commit f885e76 into ember-learn:master Mar 29, 2019
@knownasilya knownasilya deleted the feat/sticky-headers branch March 30, 2019 00:03
toddjordan added a commit to toddjordan/ember-api-docs that referenced this pull request Aug 23, 2019
sivakumar-kailasam added a commit that referenced this pull request Aug 24, 2019
Revert "feat: sticky method, property, and event headers (#336)"
Gaurav0 pushed a commit to Gaurav0/ember-api-docs that referenced this pull request Sep 16, 2019
…d_yarn/eslint-5.11.0

Bump eslint from 5.10.0 to 5.11.0
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.

5 participants