Skip to content
This repository has been archived by the owner on Jul 29, 2019. It is now read-only.

Browser IE 11 and Chrome not responding for long labels #3417

Open
goldymanikoth opened this issue Sep 5, 2017 · 8 comments
Open

Browser IE 11 and Chrome not responding for long labels #3417

goldymanikoth opened this issue Sep 5, 2017 · 8 comments

Comments

@goldymanikoth
Copy link

goldymanikoth commented Sep 5, 2017

I am using Angular2 VIS to create Network nodes and edges dynamically from my datasource using a for loop. I have noticed a weird issue when my lable size is more than 10 and having some special characters like ',' then the browser stop responding after few sec.

node sample
var nodeC = { id: cnt, label: this.sourceMappingModel.ChildFeeds[i].ChildFeedName, x: -150, y: -150 }

edge sample
var edgC = { 'from': cnt, 'to': 1, 'arrows': 'to', 'physics': false, 'smooth': { 'type': 'cubicBezier' } };

@goldymanikoth goldymanikoth changed the title Browser IE 11 and Chrome Note responding for loang labels Browser IE 11 and Chrome Note responding for long labels Sep 5, 2017
@wimrijnders
Copy link
Contributor

Would you mind posting an example label text where this happens? I'd like to see if I can reproduce it on my machine.

@wimrijnders wimrijnders changed the title Browser IE 11 and Chrome Note responding for long labels Browser IE 11 and Chrome not responding for long labels Sep 5, 2017
@goldymanikoth
Copy link
Author

try below sample labels
15.3X/NO BZ
ERGON 2 HYDROTREATED NAPHTHA,TBA,TBDS,TCB-MB BLEND
BIG WEST NAPHTHA,TBA,TBDS,TCB-MB BLEND
ERGON 2 NAPHTHA,TBA,TBDS,TCB-MB BLEND

@wimrijnders
Copy link
Contributor

Can't reproduce. Nothing special is happening with latest stable v4.20.1 or the develop version.
What version of vis.js are you using?

To be fair, I'm not using Angular to test.

@goldymanikoth
Copy link
Author

goldymanikoth commented Sep 5, 2017

4.18, do you have a vis.d.ts file available for 4.20.1 version of vis ?

@wimrijnders
Copy link
Contributor

wimrijnders commented Sep 5, 2017

Use this to test:

  <script type="text/javascript" src="https://cdnjs.cloudflare.com/ajax/libs/vis/4.20.1/vis.js"></script>

...oh wait, you're using a typescript interface definition. Crap. @mbroad, help?
Actually, you should be OK if you can manage to slip in given version. The API change is minimal.

@wimrijnders
Copy link
Contributor

wimrijnders commented Sep 5, 2017

However, on my side I can test 4.18 of course, to see if it's reproducible. Will do this.

Update: Working fine. Angular, then. Is it possible to set up a demo on jsfiddle or comparable?

@wimrijnders
Copy link
Contributor

Possibly a duplicate of #3429.

@wimrijnders
Copy link
Contributor

wimrijnders commented Oct 13, 2017

@goldymanikoth vis.js v4.21.0 has just been released. Would you mind checking that the labels render as expected now?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants