Closed angerossi closed 9 months ago
I wonder why the tests/lints failed? I was sure to run both prettier and npm test before making the PR
Thanks. This looks like the core functionality is really there. Before we can merge though I think there are several points we need to address:
I wonder why the tests/lints failed? I was sure to run both prettier and npm test before making the PR
This is unrelated to your code, it was an issue with npm I fixed on master. You should be able to rebase & get the fix.
Will this just affect the info bar or the coloring of the atoms as well? latter might be nice if it's not in here.
This is coloring the atoms with atomic properties, and doing nothing with the info panel.
Just a note that this fixes #118
I tried to merge the shape work, but it looks like this branched off main quite some time ago, so it's not entirely trivial. Would be good to rebase before doing any more work, as it won't get easier to bring the branches in sync.
I will close this PR that has apparently been superseded by #303 . If there is a reason to keep it open we can always re-open it.
Allow the atom-type properties to be displayed in the structure viewer using the RWB and Sinebow color maps supported by 3dmol.
UI changes:
Future improvements:
Other notes: