Closed JenkeScheen closed 3 years ago
I really don't want to sound like a nitpicker here, I like the work done.
But, any particular reason to not use the bootstrap modals (https://getbootstrap.com/docs/4.0/components/modal/) or alerts (https://getbootstrap.com/docs/4.0/components/alerts/), and use instead the javascript "alert"?
no reason really, just ignorance. will patch this when I find the time!
Don't worry, I'll do it. Just didn't want to mess with your work.
Great, thanks! Learning something new every day..
👍👏👏
Sent from my Huawei phone
-------- Original message -------- From: Xabier Bello @.> Date: Tue, 20 Apr 2021, 19:18 To: GPCR-ModSim/qfepweb @.> Cc: Subscribed @.***> Subject: Re: [GPCR-ModSim/qfepweb] feature group selection button (#4)
Don't worry, I'll do it. Just didn't want to mess with your work.
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/GPCR-ModSim/qfepweb/pull/4#issuecomment-823460149, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABU6HHRGLABEJB6T5WSOUTLTJWZNJANCNFSM43IEUVYQ.
När du har kontakt med oss på Uppsala universitet med e-post så innebär det att vi behandlar dina personuppgifter. För att läsa mer om hur vi gör det kan du läsa här: http://www.uu.se/om-uu/dataskydd-personuppgifter/
E-mailing Uppsala University means that we will process your personal data. For more information on how this is performed, please read here: http://www.uu.se/en/about-uu/data-protection-policy
Done.
implemented a "Save Group" and enabled
multiselect : true
in visjs code such that now:Bonus: fixed bug that @esguerra ran into that while editing an edge connectivity you can connect the edge to its own starting node. Disabled
editEdge
to handle this -> we don't want the user to be able to move edges across the network anyway because in this way edge information (such as ddG) is also transferred across the network.