m2ms / fragalysis-frontend

The React, Redux frontend built by webpack
Other
1 stars 1 forks source link

The very first molecule for the first tag should be selected #694

Open boriskovar-m2ms opened 2 years ago

boriskovar-m2ms commented 2 years ago

The very first molecule for the first tag should be selected when user comes from the landing page. Currently it's not the case and some random molecule is chosen.

rsanchezgarc commented 2 years ago

It works for cases with Tags, but it does not for the targets with no tags

boriskovar-m2ms commented 2 years ago

Could you please provide me with a name of such a target?

rsanchezgarc commented 2 years ago

Most of the targets indeed. Examples

https://fragalysis.xchem.diamond.ac.uk/viewer/react/preview/target/PGN_RS02895PGA https://fragalysis.xchem.diamond.ac.uk/viewer/react/preview/target/STAG1A https://fragalysis.xchem.diamond.ac.uk/viewer/react/preview/target/Mac1

In those cases, no compound is selected in the LHS

boriskovar-m2ms commented 2 years ago

OK problem was that we are selecting first molecule of the first tag and there are no tags so solution seems trivial 'just take first molecule' which is tricky special case (because of the implementation) but should be fixed now. Going to merge it and push to the production.