-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature request: include link to an app jnlp, if available #22
Comments
Like this? |
Yep, perfect. The only problem is that when one clicks on this jnlp link it immediately disappears -- tested in Safari and Chrome. I think it either should be there until SAMP icon appears, or you can change link to just inactive info text, something like "Loading Aladin…". The last idea is more prone to errors as launching jnlp may require further user actions and if users fails to launch it in the end he/she will have always this strange message about loading application. So better leave active link until SAMP icon appears. |
You can see your code in action at http://ocl.sai.msu.ru |
Now it "freezes" on 5 sec with "Waiting..." text, after that you can try launch again
Very nice changes! Couple of comments:
As usual, you can see what I describe at http://ocl.sai.msu.ru, I update it after your changes (it's running master branch). |
Moscow State University collaborators ask about possibility to include links to jnlp of the applications (specifically Aladin) to the VO menu when these apps aren't yet launched and have jnlp available. Once the app is launched change its representation in the VO menu to the default one (just like it is now).
The text was updated successfully, but these errors were encountered: