-
Notifications
You must be signed in to change notification settings - Fork 16
DM-6500: irsaviewer conversion #102
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
Conversation
for details: https://jira.lsstcorp.org/browse/DM-6500 also made these changes: remove irsa footer from fireflyviewer filter by selected rows auto-correct table’s filter input. multiple columns sort via sortByCols fix menu item not showing selected ife automatically builds firefly.
Some issues I've noticed when testing irsaviewer:
Not directly related to this ticket (we might need separate tickets for these):
|
The 'Read me!' link is known issue, is a bug in the server when getting Please, raise ticket and fix. On 6/14/16 10:14 AM, Tatiana Goldina wrote:
Emmanuel Joliet |
@@ -310,7 +304,7 @@ export function dispatchOnAppReady(callback) { | |||
/*---------------------------- EXPORTED FUNTIONS -----------------------------*/ | |||
export function isAppReady() { | |||
return get(flux.getState(), [APP_DATA_PATH, 'isReady']) && | |||
get(flux.getState(), [APP_DATA_PATH, 'gwtLoaded']); | |||
(get(window, 'firefly.noGWT') || get(flux.getState(), [APP_DATA_PATH, 'gwtLoaded'])); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just curious where firefly.noGWT is coming from.
Because we've removed GWT completely in the latest version, I needed a flag to decide whether or not to wait for GWT's init phase.
set min-width replaced with an IRSA's icon fix xyplot and image overlay on table sort.
for details: https://jira.lsstcorp.org/browse/DM-6500
also made these changes:
remove irsa footer from fireflyviewer
filter by selected rows
auto-correct table’s filter input.
multiple columns sort via sortByCols
fix menu item not showing selected
ife automatically builds firefly.
To test, make sure you pull ife repos as well.
same branch name on ife.