-
Notifications
You must be signed in to change notification settings - Fork 480
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
Just being honest #20713
Just being honest #20713
Commits on Feb 17, 2018
-
Replace this editorless-hack...
...with setting an editor enum or (even better) inject an appropriate editor-adaptor.
Configuration menu - View commit details
-
Copy full SHA for df0e2ed - Browse repository at this point
Copy the full SHA df0e2edView commit details -
This getValue() call is a workaround...
...for a Droplet bug, See droplet-editor/droplet#137 Calling getValue() updates the cached ace editor value, which can be out-of-date in droplet and cause an incorrect early-out. Could remove this line once that bug is fixed and Droplet is updated.
Configuration menu - View commit details
-
Copy full SHA for 846e6b5 - Browse repository at this point
Copy the full SHA 846e6b5View commit details -
Configuration menu - View commit details
-
Copy full SHA for 301e4c4 - Browse repository at this point
Copy the full SHA 301e4c4View commit details -
Configuration menu - View commit details
-
Copy full SHA for a50ac8a - Browse repository at this point
Copy the full SHA a50ac8aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 48e6a64 - Browse repository at this point
Copy the full SHA 48e6a64View commit details -
This sequencing feels backwards...
...Eventually, we ought to preview and get dimensions from the local filesystem, async with the upload itself, but that will mean refactoring away from the jQuery uploader.
Configuration menu - View commit details
-
Copy full SHA for b6611cd - Browse repository at this point
Copy the full SHA b6611cdView commit details -
Configuration menu - View commit details
-
Copy full SHA for b8e4a68 - Browse repository at this point
Copy the full SHA b8e4a68View commit details -
Configuration menu - View commit details
-
Copy full SHA for be367b9 - Browse repository at this point
Copy the full SHA be367b9View commit details -
Configuration menu - View commit details
-
Copy full SHA for d87e832 - Browse repository at this point
Copy the full SHA d87e832View commit details -
This whole file should go away...
...when we have a shared Javascript User object that can be available on page load.
Configuration menu - View commit details
-
Copy full SHA for 5442f10 - Browse repository at this point
Copy the full SHA 5442f10View commit details -
Configuration menu - View commit details
-
Copy full SHA for 3c76bad - Browse repository at this point
Copy the full SHA 3c76badView commit details -
...more likely to ensure it's unique and fits within 48 characters. Maybe grab this MIT-licensed implementation via node? https://github.com/blueimp/JavaScript-MD5
Configuration menu - View commit details
-
Copy full SHA for 349152a - Browse repository at this point
Copy the full SHA 349152aView commit details -
Do we benefit from inheritance here? Would it be cleaner to make this not-an-entity that manipulates a stock NetSimClientNode? Will another developer find it easy to understand how this class works?
Configuration menu - View commit details
-
Copy full SHA for d4be723 - Browse repository at this point
Copy the full SHA d4be723View commit details -
Modify and return the template node...
...instead of making two in this method.
Configuration menu - View commit details
-
Copy full SHA for 9f5f0b6 - Browse repository at this point
Copy the full SHA 9f5f0b6View commit details -
...and ticking the routers up to netsim.js (or elsewhere)
Configuration menu - View commit details
-
Copy full SHA for c43ce77 - Browse repository at this point
Copy the full SHA c43ce77View commit details -
...so we can be notified when writes succeed instead of making a 50ms guess, and make this a properly async method.
Configuration menu - View commit details
-
Copy full SHA for 7934cb6 - Browse repository at this point
Copy the full SHA 7934cb6View commit details -
This is to expose methods we need...
...in the global namespace for testing purpose. Would be nice to eliminate this eventually.
Configuration menu - View commit details
-
Copy full SHA for 4dd1725 - Browse repository at this point
Copy the full SHA 4dd1725View commit details -
Configuration menu - View commit details
-
Copy full SHA for 21e59b8 - Browse repository at this point
Copy the full SHA 21e59b8View commit details -
Configuration menu - View commit details
-
Copy full SHA for 8a70924 - Browse repository at this point
Copy the full SHA 8a70924View commit details