vue-next-select icon indicating copy to clipboard operation
vue-next-select copied to clipboard

Close on select not working when upgrading from 2.5 to 2.6

Open eboye opened this issue 3 years ago • 6 comments

eboye avatar Jun 13 '21 20:06 eboye

@eboye It works for me. Could you please provide more details or a reproduce link?

I have create the following sandbox for this issue, you could fork it.

To Reproduce A reproduce link: https://codesandbox.io/s/vue-next-select-issue-219-ppizx?file=/src/views/BasicUsage.vue

Steps to reproduce the behavior:

  1. Go to Basic Usage.
  2. Click on select.
  3. Click on first option.

Expected behavior Select should be closed.

soc221b avatar Jun 14 '21 03:06 soc221b

Hmm, I kind of fixed it ... I was not using CSS from the package but replicated it in my own scss so it could be styled with the design of the application.

Have you altered your css and possibly the way the dropdown is shown/hidden?

It would be helpful if you could provide this kind of "breaking changes" in the release notes.

eboye avatar Jun 14 '21 22:06 eboye

Thank you for your reply.

I think it is caused by this commit.

But I need to consider whether I should revert it and whether it is a BREAKING CHANGE 🤔 .

soc221b avatar Jun 15 '21 05:06 soc221b

Yup, that's the one :D

I've added this to make it work image

As I'm using tailwind in my project I've extracted all the css from all (not many) libraries to the scss so the css footprint could be lower.

eboye avatar Jun 15 '21 16:06 eboye

Are the datasets and a11y attributes sufficient for all of your use case?

BTW, I would recommend you to change .vue-dropdown[data-is-focusing='false'] to .vue-select[aria-expanded='true'] vue-dropdown since the second one is more formal, and I will remove some dataset (if it can also be detect by aria-* attributes) in next major version too.

soc221b avatar Jun 16 '21 08:06 soc221b

Yup they are, I just needed something to easily style the select and have it work with keyboard navigation and not do everything from scratch as I'm using tailwind and very little plugins ...

Sure, I'll switch to aria attrs. Good point.

eboye avatar Jun 19 '21 21:06 eboye