#137 Cannot select a structure from ChemSpider

pending-postponed
rscdaya
None
6
2014-07-28
2014-01-21
Tim Parkinson
No

I was trying out Aileen's structure importer on Aurora, currently at r701.

I tried to make a new post or edit an old post and attempted to insert a structure from ChemSpider.
When I click on the ChemSpider button in the TinyMCE window then the ChemSpider menu appears, I select Insert compount information and accept all defaults.
Then I click Go then I accept the default search of c4h6 and Search. Ten records are returned with pictures in the results box.

Only the boxes in the first column are clickable and when I click one of them, nothing happens (I am not returned to the notebook entry with my chosen structure).

This happens on both IE11 and Chrome.
Is this a config issue or an incomplete code issue (waiting for the latest merge) or a bug?

Discussion

1 2 > >> (Page 1 of 2)
  • Tim Parkinson
    Tim Parkinson
    2014-01-21

    Only the first column of structure images is clickable.

     
  • Tim Parkinson
    Tim Parkinson
    2014-01-21

    And I have just seen the same behaviour this morning on Aurora r712 which now allows drawing but the insert behaviour is the same. (So Dave is probably right that it's an issue with the ChemSpider service).

     
  • Tim Parkinson
    Tim Parkinson
    2014-01-21

    My mistake. Aurora has now move up to r714.

     
  • Tim Parkinson
    Tim Parkinson
    2014-01-21

    I thought it might be something to do with having more than one search result or more than one line of results.

    So I just tried to reproduce the Youtube example - inserting benzene.
    I copied the video, searching for benzene, only one record returned from the search.
    That Benzene ring thumbnail square is clickable (there is a hand if you hover over it) but
    clicking it does not return me to my Notebook entry as it did in the video.

    So perhaps this is something to do with the merge to the trunk after all as it used to work?

     
    • rscdaya
      rscdaya
      2014-01-22

      OK I can reproduce the bug about nothing past the first column of results being clickable (it looks like a general problem with my widget so I'll see if I can fix it)... The Benzene bug looks like it's a separate issue - it's with a separate widget (the search one not the editstoichiometry table) although it looks the same. Seems to work fine on my local copy so that pne might well be to do with the merge to trunk... i'll see if there's anything obvious going on with the files...

       
  • rscdaya
    rscdaya
    2014-01-22

    I tracked down the bug with the "Insert info from ChemSpider" widget (it only showed up in some browsers which is why it was only showing up sometimes) and committed revision 725 to the labtrove code to fix that bit. Like I say, the other bit about only being able to select the first column of search results is a separate issue with the widget itself on the chemspider server. So I'll look into that one but it won't require a change to the labtrove code to fix it (the labtrove code can still go through to liberata)...

     
  • Tim Parkinson
    Tim Parkinson
    2014-01-23

    Aileen,

    I confirm that those search results that are clickable can now be clicked and are now inserted into the LabTrove entry so the serious part of this issue is now fixed.

    Dave has spotted what is happening about the non-clickables.
    It's to do with available room for the extra information pop-up box.

    If you use the default search (c4h6) it returns ten matches.
    These are laid out 4-4-2 in the default sized window (at least on my machine).
    Hover over each of the leftmost 3 column entries. The info box pops up to the right.
    Now hover over any of the other seven.
    There is no room for the pop-up box to the left or right
    without obscuring the cursor.
    So the box does not appear, and it looks as though the item cannot be clicked unless that
    box is on show.

    We can prove this by resizing the search results window which results in
    more or less clickable items. In extremis, if the search results window is made tall and thin then there is no room for the pop-up anywhere and no results can be selected.
    If on the other hand the search results window is made short and wide, then there is room
    for a box either to left or right and any result can be selected.
    So there is a workaround.

    I agree that this is an issue with ChemSpider rather than LabTrove and it can be fixed
    in parallel and doesn't block us from updating Liberata with the current tip of the trunk.
    I'll leave this ticket open until ChemSpider resolves the selectability issue.

     
  • rscdaya
    rscdaya
    2014-01-23

    Hello Tim,
    I've looked into this and see that yes you're right - it is to do with the window sizing as it's possible to resize / scroll around the window and then be able to click on them...
    I see also that it's an issue that's been picked up and fixed in version 2 of the ChemSpider widgets - we're currently on version 1 but I'm going to upgrade to version 2 in the next phase of work (which will involve all the development that arises from the feedback of this trial). I'm not intending to do that upgrade process at this point as it's a big change and should be done carefully and properly tested. Given that it has been fixed in the next version, would it be possible to mark this as a known issue for now and I'll make sure that in the next phase of development? It's just that this would involve quite a few files being back-published to a previous version of the widgets on our live site which again could have unforeseen consequences... I know it's not ideal, but I think it would be the safest option at this stage if we can all live with it...

     
  • Tim Parkinson
    Tim Parkinson
    2014-01-24

    Agreed, we can live with it for now.
    We don't seem to have a 'known issue' status so I've switched this ticket to 'postponed'
    until ChemSpider v2 is up.

     
  • Tim Parkinson
    Tim Parkinson
    2014-01-24

    • status: open --> pending-postponed
     
1 2 > >> (Page 1 of 2)