ngx-chips icon indicating copy to clipboard operation
ngx-chips copied to clipboard

typeahead results Angular 8 window doesn't show in IE 11 or Edge, display:none is never updated

Open daddyschmack opened this issue 6 years ago • 4 comments
trafficstars

PLEASE MAKE SURE THAT:

  • you searched similar issues online (9/10 issues in this repo are solved by googling, so please...do it)
  • you provide an online demo I can see without having to replicate your environment
  • you help me by debugging your issue, and if you can't, do go on filling out this form

I'm submitting a ... (check one with "x")

[ x] bug report => search github for a similar issue or PR before submitting
[ ] support request/question

Notice: feature requests will be ignored, submit a PR if you'd like

Current behavior

when I click in the field, the typeahead window doesn't show

Expected behavior

the typeahead window should be visible

Minimal reproduction of the problem with instructions (if applicable)

What do you use to build your app?. Please specify the version

Angular-cli

Angular version:

Angular 8.0

ngx-chips version:

2.0.2

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

IE 11, Edge example site won't load in ie11, either: https://angular-mfppay.stackblitz.io/

daddyschmack avatar Jul 03 '19 20:07 daddyschmack

I think it's the same issue as it was in Safari. I fixed it in the dependency of this package earlier, but it waits for merging and bumping of versions. https://github.com/Gbuomprisco/ng2-material-dropdown/pull/50 https://github.com/Gbuomprisco/ngx-chips/issues/877

dmitrynop avatar Jul 07 '19 12:07 dmitrynop

Unless I missed something, It didnt solve the same way. We had to override the style by setting a case property with atag with I display flex instead of display none, and then it works just fine

On Sun, Jul 7, 2019, 6:48 AM Dmitry Nikitin [email protected] wrote:

I think it's the same issue as it was in Safari. I fixed it in the dependency of this package earlier, but it waits for merging and bumping of versions. Gbuomprisco/ng2-material-dropdown#50 https://github.com/Gbuomprisco/ng2-material-dropdown/pull/50 #877 https://github.com/Gbuomprisco/ngx-chips/issues/877

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Gbuomprisco/ngx-chips/issues/885?email_source=notifications&email_token=AAVAIY2CIFPD2A5KPO7P5MLP6HQ3FA5CNFSM4H5J7G3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZLK4KQ#issuecomment-508997162, or mute the thread https://github.com/notifications/unsubscribe-auth/AAVAIY6G7NVQJIHEEKJZDQ3P6HQ3FANCNFSM4H5J7G3A .

daddyschmack avatar Jul 08 '19 02:07 daddyschmack

Unless I missed something, It didnt solve the same way. We had to override the style by setting a case property with atag with I display flex instead of display none, and then it works just fine

Hi, could you show me how did you make it work on Safari please?

Here's my template (working well with Chrome)

<tag-input class="hashtags" [(ngModel)]="object[field.key]" [onlyFromAutocomplete]="true" [displayBy]="field.selectedItemDisplayBy" [identifyBy]="field.selectedItemKey"> <tag-input-dropdown [showDropdownIfEmpty]="false" [displayBy]="field.selectedItemDisplayBy" [identifyBy]="field.selectedItemKey" [autocompleteItems]="field.selectOptions"> </tag-input-dropdown> </tag-input>

Looking forward to hearing from you

azuxx avatar Aug 02 '19 12:08 azuxx

I'm also encountering the same issue, the typeahead dropdown doesn't appear on classic Edge browser, it shows a small blank dropdown for a split second if I set showDropdownIfEmpty = true. I've tried copied and pasted code from the demo page https://github.com/Gbuomprisco/ngx-chips/blob/b6a39d368eab9198c6c926e838806b168c9cceb7/demo/home/home.ts but still no luck

I'm using Angular 8 and ngx-chips 1.8.2, unfortunately I can only use ngx-chips 1.8.2. @daddyschmack can you explain your solution again, I am still confused

ZacVND avatar Apr 21 '20 12:04 ZacVND