Skip to content
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

VUE3 misc KvLightbox fixes #168

Merged
merged 6 commits into from
Jan 28, 2022

Conversation

ryan-ludwig
Copy link
Contributor

@ryan-ludwig ryan-ludwig commented Jan 28, 2022

Adds vueuse package for focus lock. Couldn't get the new version of vue-package-lock to work 🤷

Fixes a few other things lost in the conversion. If time, some tests around these things would be better.

@@ -30,7 +30,12 @@ module.exports = {
}
],
include: path.resolve(__dirname, '../'),
})
}),
config.module.rules.push({
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Resolves an issue I had after importing vue-use. vueuse/vueuse#718

Copy link
Collaborator

@emuvente emuvente left a comment

Choose a reason for hiding this comment

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

Nice fixes! I'll look into good ways to test some of these things

@ryan-ludwig ryan-ludwig merged commit 3681919 into vue3_kvlightbox_VUE-892 Jan 28, 2022
@ryan-ludwig ryan-ludwig deleted the vue3_kvlightbox_VUE-892_fixes branch January 28, 2022 17:24
} from 'vue-demi';
import { mdiClose } from '@mdi/js';
import FocusLock from 'vue-focus-lock';
import { useFocusTrap } from '@vueuse/integrations/useFocusTrap';
Copy link
Collaborator

Choose a reason for hiding this comment

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

Glad you found a replacement!

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.

3 participants