-
Notifications
You must be signed in to change notification settings - Fork 1
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
fix: search bar uses all width in mobile #474
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/platforme/ripe-components-vue/2TkfiRKTVPvnnmCZZYDyoNk1WuY7 |
Tests failing due to new release of vue-test-utils https://github.com/vuejs/vue-test-utils/releases |
@BeeMargarida what commit caused the regression? |
The regression this PR fixes or the one that caused the tests to be failing (this is not a commit, its an update of a dependency)? |
The regression this PR fixes was caused by #449, where instead of using the search, the tooltip component now surrounds the search bar. |
@BeeMargarida @gcandal I'm "debumping" packages to avoid the |
select
to start failing. Issues about the regression: vuejs/vue-test-utils#1820 & vuejs/vue-test-utils#1808 (the PR that possibly caused this regression). vuejs/vue-test-utils#1824