react-fonticonpicker icon indicating copy to clipboard operation
react-fonticonpicker copied to clipboard

Modal closes automatically when switching to first or last icon page with button

Open GuentherK opened this issue 5 years ago • 8 comments

First i would like to thank you for your effort creating this package.

We have the issue that the icon picker modal closes automatically when you switch vom the 2nd page to the first page or when you switch from the length -1 to last page.

How to reproduce this issue:

just open the icon picker from your demo page https://fonticonpicker.github.io/react-fonticonpicker/examples/

this is very problematic if you have only 2 pages. E.g on your demo page the 'implied search' example with the 'Web Application Icons' filter

GuentherK avatar May 23 '19 17:05 GuentherK

Hi @Greg-mso ! Would it be possible to release a new version with this fix? It's quite annoying preventing us to actually use it, so it would be really great :) Thanks a lot for your work!

obrejla avatar Mar 05 '20 14:03 obrejla

Hi @obrejla , it was merged into master by @swashata on 5th Jan this year. You should be able to update to the latest version and see the fix.

Greg-G23 avatar Mar 05 '20 14:03 Greg-G23

@Greg-mso Unfortunately, there is version 1.2.0 on npm - https://www.npmjs.com/package/@fonticonpicker/react-fonticonpicker More than year old.

obrejla avatar Mar 05 '20 14:03 obrejla

@obrejla Afraid I don't have permission to do so, however you could update your package.json to use '/fontIconPicker/react-fonticonpicker#master' and it should pick up those merged changes. Hope that helps.

Greg-G23 avatar Mar 05 '20 15:03 Greg-G23

Hi @Greg-mso , imho if @swashata is the main maintainer, then he could do the regular publish of new fixed version? @swashata , what do you think? Thanks.

obrejla avatar Mar 06 '20 06:03 obrejla

Hi all, the thing is there are lots of errors in the repo right now, because when I made it, I didn't have proper understanding of react (and the life cycles). I intend to rewrite this whole thing with hooks. Till then you can fork, build and use your own version of the whole thing. I am sorry I don't have the time budget right now, but I will try to in a couple of months.

swashata avatar Mar 06 '20 14:03 swashata

/fontIconPicker/react-fonticonpicker#master

@obrejla Afraid I don't have permission to do so, however you could update your package.json to use '/fontIconPicker/react-fonticonpicker#master' and it should pick up those merged changes. Hope that helps.

@Greg-mso - can you show an example of what your package.json line looks like to pick up #master ?

gregfenton avatar Apr 29 '20 18:04 gregfenton

Anything new with this issue? I have the same issue :(. The panel close when I click to the next or previous page...

anisayari avatar Aug 05 '20 09:08 anisayari