Limit with omitted

[dany-fu]

Issue #53 and #73
Renders 30 glyphs max, the 31st glyph is the omitted glyph

[cjmyers]

So, this actually will not work with SynBioHub. SBH does not use browser.js, but goes directly to getDisplayList. The addition of the omitted detail and the pruning of the extra segments should be done in getDisplayList.

[cjmyers]

This does not break anything, so it is fine to be merged, but I will leave this branch open, and a new pull request on this branch can be submitted later, which moves this functionality into getDisplayList.

[dany-fu]

Here, we are using the code that Zach had committed a while back for limiting, which is to slice the displayList. I closed the pull request after Arezoo raised the issue of how that could impact the interactions that she's currently working on, and it's also not an ideal long term solution if in the future we want to consider something like an expansion or some other way of rendering the rest of the glyphs. I think we should not limit the displayList but make the cut off during the rendering instead

[cjmyers]

Agreed. I will revert this one.

…

On Jul 11, 2018, at 1:10 PM, Dany ***@***.***> wrote: Here, we are using the code that Zach had committed a while back for limiting, which is to slice the displayList. I closed the pull request after Arezoo raised the issue of how that could impact the interactions that she's currently working on, and it's also not an ideal long term solution if in the future we want to consider something like an expansion or some other way of rendering the rest of the glyphs. I think we should not limit the displayList but make the cut off during the rendering instead — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#79 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ADWD9860mzzlaK2LkPQHsXa3rIP5twWdks5uFk22gaJpZM4VKODb>.