Skip to content
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

Fixes to workspace.js #130

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

baptmont
Copy link

The commit contains fixes to the following issues found when using the workspace screen:

  • Error in files with multiple pages, where only some had columns. (fix in line 107 changed push to concat)
  • Columns loaded from an existing rule were offset (fix in line 166 by removing half the width of the div to compensate for operation at line 219 and correct offset)
  • By using expressions like $("#flag-size-s").val() ? true : false a rule with a value of false will be set to true. This happens since the expression simply checks if the textbox has a value and when an existing rule is loaded the value in the textbox is set to the string "false"; changed to $("#flag-size-s").val() == "true". This requires the value in the textbox to be exactly the string "true".
  • If no flavor for the algorithm was selected an alert would be shown but the ajax request to '/jobs' would go through redirecting to a different page (fixed by changing alert to throw an exception and perform alert in a try-catch that covers the ajax request).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant