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

Move searchbox to scroller element so it doesn't hide the scrollbar #2969

Merged
merged 1 commit into from May 16, 2017

Conversation

Projects
None yet
2 participants
@vlakoff
Contributor

vlakoff commented May 15, 2016

On a related note, I also tried moving this searchbox to the content element, but it jitters when scrolling.

Still, the scroller element seems to be the perfect place.

@nightwing

This comment has been minimized.

Member

nightwing commented May 16, 2017

i am not completely sure if this is backwards compatible, but seems to be worth it. merging

@nightwing nightwing merged commit 9200b31 into ajaxorg:master May 16, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@vlakoff vlakoff deleted the vlakoff:searchbox branch Jan 14, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment