git review:
single commit with appropriate message.
Needs to be rebased on master.
Code review:
Changes are simple and intuitive.
Need to check: is line 239 the only time that the loadData method is called? --> yes, it is a private method, called exactly once in this class.
Functional review:
This has been challenging. Just in testing with 9.0.1, IGB is not responding to links the way I think it should.
With my example link in the last comment and with the example given with the issue, IGB 9.0.1 gave me a message saying that I should zoom in a load data. On Sneha's branch, I did not get that message using those links. However I do get the message using the Galaxy View link which makes sense because it does not contain start and end values.
____________________________
Overall, this looks good. Before merging, I think Ann should consider weather or not this is a good idea.
This removes an inconvenience for advanced users. We already have a preferences option, so if you find the box annoying you can check the box to never see it again.
However, the general scenario of a track appearing, and being tiny and users not knowing that they need to load data (or zoom in and load data), is why we have that message in the first place. Many new users will be in exactly that position when they use BAR or Galaxy and are directed to use IGB for the first time. Even if the coordinates (start and stop) are specified, they may still specify a wide region. If the data is not loaded automatically, then the track will be tiny, and easy to not notice. See the image I attached.
Tested on Windows environment, it is working fine. I tried loading a bookmark and it zoomed in to the location where bookmark was saved. No "zoom in and click load data" message shown.