Commit graph

93 commits

Author SHA1 Message Date
Karishma Chadha
28ee2a6aef
Lint (fix long line)
Co-Authored-By: towerofnix <9948030+towerofnix@users.noreply.github.com>
2018-12-13 12:38:00 -04:00
Florrie
273d5c2f7e Use CLOUD_PREFIX.length instead of assuming it 2018-12-12 15:33:01 -04:00
Karishma Chadha
799c7bd748
Don't include cloud symbol in default value of "rename variable" prompt
Co-Authored-By: towerofnix <9948030+towerofnix@users.noreply.github.com>
2018-12-12 15:30:05 -04:00
Florrie
30b2443ab2 Use variable name as rename prompt's default value 2018-12-06 22:31:26 -04:00
Paul Kaplan
2a0bd8be37 Fix the "Create Variable" button in the playground by providing default 2018-12-04 15:54:06 -05:00
Karishma Chadha
93bbd0b00f Track cloud variable info and handle naming and renaming cloud variables. 2018-11-13 16:43:00 -05:00
Karishma Chadha
d9a079d550 Fix issue with renameVariable which didn't know about additionalVars. Decide whether or not to use additionalVars based on the new variable isLocal flag. 2018-07-12 19:15:49 -04:00
Karishma Chadha
768b5e1b09 Initialize additionalVars in case it isn't provided. This is necessary for the vertical playground for example. 2018-07-06 13:03:36 -04:00
Karishma Chadha
c23fe53344 Allow for local variables by passing in extra scope information (provided by the gui). This information is included in the varCreate event. Pass in a list of additional var names to check against when creating a variable (e.g. local var names from all sprites when checking for name conflicts to create a global var. 2018-07-03 14:01:49 -04:00
Rachel Fenichel
af4060c75b Add ordered option to getDescendants and all uses.
"
2018-05-16 12:51:41 -07:00
Karishma Chadha
c01f92c217 Adding variable type info to blockly prompt so scratch-gui can pick it up. 2018-01-29 18:23:49 -05:00
Karishma Chadha
7464071fca Making create/rename and associated validators more permissive. 2018-01-29 10:40:07 -05:00
Karishma Chadha
7a63cb1237 Fixing comments/typos. 2018-01-26 09:33:22 -05:00
Karishma Chadha
8a8156aea6 Cleanup validators. Removing recursive calls to bring up prompt again. This was already not happening in GUI, and this new behavior is consistent with Scratch 2.0. 2018-01-25 10:44:39 -05:00
Karishma Chadha
20d1c1f028 Break out variable validation into separate functions by type. Other refactoring/cleanup, especially w/regards to trimming names of user-provided input). Variables/Lists/Messages now all have separate alert messages and modal titles by type for both creation and renaming of a variable. 2018-01-25 10:44:39 -05:00
Karishma Chadha
44a56b32dd Some code cleanup. Removing check for procedure name used when creating a new variable because scratch doesn't care if variables and procedures share names. The isNameUsed function was basically a no-op here because Blockly.Procedures doesn't have a getProcedureDef function defined in scratch-blocks. 2018-01-22 17:24:23 -05:00
Karishma Chadha
1a6842de71 Making variable names case sensitive. We don't need to check for variable type, because Scratch 2 also allows creating 'new' broadcast messages with the same name (different case) as an existing message name. E.g. if 'message1' exists, creating a new message with the name 'Message1' is allowed. Both options will show up in the dropdown menu for broadcast blocks. Both messages will evaluate to the same name. E.g. executing a 'broadcast Message1' block will trigger both 'when I receive message1' and 'when I receive Message1' hats. 2018-01-22 16:34:33 -05:00
Karishma Chadha
78b2cd84dc Cleanup of realizePotentialVar function. 2018-01-22 10:53:15 -05:00
Karishma Chadha
8b5762581b Code cleanup, removing todo comments. 2018-01-18 15:03:46 -05:00
Karishma Chadha
ec257e51ca Fixing errors from previous merge conflict resolution. 2018-01-18 15:03:46 -05:00
Rachel Fenichel
7102012240 Pulling in final changes from upstream blockly/variables_by_id branch.
Includes commits 1fa27fa (01/12/18) through fa91c6b (01/16/18).
2018-01-18 15:03:46 -05:00
Karishma Chadha
8962f2b0e4 One sentence summary.. 2018-01-18 15:03:46 -05:00
Karishma Chadha
f19c5569bf Adding JSDoc for realizePotentialVar function, explaining its use. 2018-01-18 15:03:46 -05:00
Karishma Chadha
d8a63f3aa4 Fixing issue where importing an SB2 project that uses the 'message1' default broadcast message wasn't wokring because of the way the realizePotentialVars function was written. The imported 'message1' broadcast message was conflicting with the one that is a potential variable in the flyout. 2018-01-18 15:03:46 -05:00
Rachel Fenichel
855ec2665a Pulling in the latest changes from google/blockly/variables_by_id branch,
starting with commit 6218750 (December 19, 2017) through commit 80b397f (January 10, 2018).
2018-01-18 15:03:46 -05:00
Karishma Chadha
87740cdea7 Fixing behavior where converting a potential variable into a real variable (e.g. dragging out a broadcast block out of the flyout with the default 'message1' option selected or creating a new message called 'message1' from the flyout) keeps the variable in both the potential variable map as well as the real variable map, so dropdown menus in the broadcast blocks from the flyout will display duplicate entries for the variable. 2018-01-18 15:03:46 -05:00
Karishma Chadha
6de6e9f3b9 Can create new messages now...
Broadcast messages from flyouts also work now. Re-enabling new message option from broadcast blocks in flyout.
One behavior that is different from current functionality is that 'message1' will not exist in dropdown menus outside the flyout unless a broadcast block w/'message1' selected is dragged out of the flyout
(thus creating 'message1' as a variable in the target workspace).
A problem with this is that we can currently create 'message1' as a new variable (from a block in the flyout), and then blocks in the flyout will have two 'message1' options in the dropdown. Still need to investigate a fix for this.
2018-01-18 15:03:46 -05:00
Karishma Chadha
82e4cdc624 Fixing linter error. 2018-01-18 15:03:46 -05:00
Karishma Chadha
db44603362 Pulling in changes from blockly/variables_by_id. 2018-01-18 15:03:46 -05:00
Karishma Chadha
2f8151bb90 Support for having different modal titles for new message/variable/list modals. 2017-12-22 13:47:35 -05:00
Karishma Chadha
3c53fbd2b5 Small code cleanup. 2017-12-21 16:23:03 -05:00
Karishma Chadha
7a305032d6 Addressing PR comments. 2017-12-21 15:12:11 -05:00
Karishma Chadha
41ede2fe2d First cut at allowing whitespace message names. Rendering for the dropdown menu option for the new whitespace message is still off. 2017-12-21 14:56:42 -05:00
Karishma Chadha
a16fc794c2 Making 'list' specifiying variable type for list blocks a constant instead of a hardcoded value. Refactoring existing code to use this new constant. Updated name of constant referring to broadcast message variable type to make it more specific/match new name convention for variable type names. 2017-11-21 13:49:47 -05:00
Karishma Chadha
208d1892fc Fix for 2017-11-21 09:05:00 -05:00
Karishma Chadha
f84e764951 Adding some TODO comments referencing existing issues also discussed in this PR. 2017-11-20 17:15:04 -05:00
Karishma Chadha
6eafe6ed51 If a field dropdown is used to create a new message, the field then gets updated with the name of the newly created message. There are also some todo comments here for future work that has been documented in two github issues ( and )
.
2017-11-17 10:49:46 -05:00
Paul Kaplan
807743d20f Change variableType to variabletype in xml.
Scratch-blocks version of blockly pr 
2017-11-06 10:29:43 -05:00
Paul Kaplan
db865f24f4 Remove unused variable category code.
It had been moved to the data_category.js file. It is dead code now.
2017-10-31 09:34:19 -04:00
Rachel Fenichel
eed4e33f84 Escape variable names correctly when serializing to XML () 2017-10-19 12:02:42 -04:00
marisaleung
bf7f8bbaeb Adding lists! 2017-07-25 15:03:40 -07:00
marisaleung
ffc1eb7682 Add VariableType checks. 2017-07-21 11:02:25 -07:00
DD Liu
c0e1335a3d make variable id match block id 2017-07-10 15:57:54 -04:00
Paul Kaplan
ead4916e53 Use variable field directly instead of shadow 2017-06-14 16:30:56 -04:00
Paul Kaplan
10fcaf993d Fix nits 2017-06-02 11:45:47 -04:00
Paul Kaplan
412c971d28 Set checkbox state after creating a variable through the prompt 2017-06-02 11:30:45 -04:00
Ray Schamp
e92c5f5325 Give persistent block ids to variables in the flyout 2017-06-02 09:32:44 -04:00
Paul Kaplan
dd964828e1 Merge/05 24 ()
* JSON definitions for colour blocks ()

Replaces old colour block definitions with a Blockly.defineBlocksWithJsonArray(..) call. Generator unit tests continue to load and pass, signifying compatibility with prior block definitions.

Replaces extension 'math_number_tooltip' with the reusable 'parent_tooltip_when_inline' extension, also used by colour_picker. Includes tests.

* Rewrite tree.service.js.

- Remove unnecessary code and functions.
- Add documentation where needed.
- Fix a bug arising when a block on the workspace is attached to an existing link.

* Use setValue in fieldTextInput so that procedure renaming works

* Further cleanup and removal of unnecessary functions. Pull some strings out for i18n.

* Use bindEvent_ instead of bindEventWithChecks_ for longStop

* Clean up workspace.component.js. When moving a block from one place to another, move all blocks after it too, and adjust the active descs accordingly.

* Unit tests for JSON block definitions (just the start) ()

 * Beginnings of a JSON block definition unit test set.
 * Dispose of unit test workspaces and blocks in finally blocks.
 * Clarify JSON error message by echoing arg notation.

* New blocks text_count, text_replace, and text_reverse ()

Includes generators for all languages and units tests on those generators.

* Fixing combo boxes getting out-of-sync with NVDA.

Combo boxes need to be special cased like text input. Also, Escape is
a reserved button in NVDA, so I added Enter as a way to "submit and
move up a level" in addition to escape, so these boxes can be edited
while NVDA is on.

* Temporary fix for broken text field validation.

* rebuild

* Add a block to reverse a list ()

* Localisation updates from https://translatewiki.net.

* Porting math.js blocks to JSON ()

Moving all `math.js` definitions into a single JSON array, complete with i18n syntax for all messages, dropdowns, and tooltips.

Adding Blockly.Extensions.buildTooltipForDropdown(..) to facilitate the creation and error-checking of tooltips that update based on the value of a dropdown.

Now warn on raw string in JSON 'extensions'.

* Fixing JSON support for images in dropdowns. Adding tests. ()

Fixes .

* Update README.md

Add a link to our forum.

* Correcting math_change color

* Enable custom flyout categories.

* Add some safety

* Update the set of reserved words in Python to reflect the current state of Python (2.7 and 3.6). ()

* Localisation updates from https://translatewiki.net.

* .getOptions_() to .getOptions() ()

Fixes .

* Blockly.Extensions.buildTooltipForDropdown(..): Deferred validation. ()

Defer tooltip message string check until after load, when all Blockly.Msg should be loaded.
Avoids validation in headless mode, due to lack of document.readyState.

* annotation updates

* annotation updates

* jsdoc corrections ()

* Remove use of Array.prototype.includes which is not implemented in IE or Edge < 14. Fixes .

* Attempt to work around the IE/Edge bug where `getComputedTextLength()` throws an exception when the SVG node is not visible. This workaround forces a re-render, which in turn, forces a re-calculation of the node width once a block is inserted into the workspace SVG. This workaround is only executed on IE and Edge. See https://groups.google.com/forum/#!topic/blockly/T8IR4t4xAIY for the initial discussion of this issue.

* Change CSS transforms to work with older browsers ()

* Change the setting of the CSS transform properties on SVG nodes to set both the unprefixed version and the `-webkit-` prefixed version so that Blockly correctly renders in order browsers, such as Safari < 9 and iOS Safari < 9.2. For discussion of this issue, see https://groups.google.com/forum/#!topic/blockly/o3pERaRQhSg

* Correct the separation between the CSS transform property and the rest of the CSS that was in the variable misleadingly called "transform".

* Don't try to get block position in a headless workspace

* Stop bumping neighbours in headless blockly

* Place context menu correctly on touch

* Clear all active desc ids when the 'Erase Workspace' button is pressed.

* Fix a bug where splicing a block between two linked blocks disconnects the group and messes up the focus.

* Deleting a top-level block does not cause blocks after it to be deleted. Properly handle the active desc for this case.

* Localisation updates from https://translatewiki.net.

* Use the empty field placeholder for dropdowns that do not have a value selected.

* Bugfix for . I incorrectly converted one CSS transform setting to use the cross-browser setting function in 40a063763c74b3f712c3057565966c25d5cfdb10. ()

* Adding @namespace annotations for JSDoc. ()

* Fix typo causing TypeError ()

* Pinning the angular2 dependency, and including licenses. ()

* Add skeleton for tests on rendered workspaces

* Fix some lint errors

* Localisation updates from https://translatewiki.net.

* Correct changedState in setWarningText() ()

When clearing warnings on blocks with IDs, the changedState variable should be true if the text changed. This will trigger the block being reshaped and remove the space for the notification icon (this.bumpNeighbours_).

* Adds Block.prototype.mixin() and Blockly.Extensions.registerMixin(). ()

Adds Block.prototype.mixin() and Blockly.Extensions.registerMixin().
This adds support for a common use pattern in extensions, and adds
error checking to avoid future incompatibilities.

* Porting Logic blocks to JSON ()

Extensions, mixins, mutators and constants now grouped under the new namespace Blockly.Constants.Logic.

* Improving errors/warnings with Block.toDevString() and Connection.toString(). ()

* Add isEditable to field, and add tests

* Separate tests

* Blockly.Constants.Math and Blockly.Constants.Colour extension constants ()

Also, correcting quotes in logic.js.

* Correction to logic_ternary type check ()

* Porting Loop blocks to JSON ()

 * Improved documentation on `Blockly.Extensions.buildTooltipForDropdown`
 * Replaced incorrect uses of `@mixes` JSDoc annotation (on mixin extensions) with `@augments Blockly.Block`.
 * Added Blockly.Extensions.buildTooltipWithFieldValue() extension helper.

* Workspace isDraggable

* JSONify simple list blocks

* JSONify variable blocks

* Initial text block, with a mixin to generate quote image fields. ()

Text block now uses the extension "text_quotes", supported by Blockly.Constants.Text.QUOTE_IMAGE_MIXIN.quoteField_(fieldName), so that each platform can use the best platform appropriate image (size, density, etc.) for the quotes.

* Add no-op stub .neighbors() for headless Connection.

* Adding tests for logic_ternary block in a new jsunit test framework.

* Correcting output of the logic_null block.

* Potential bug fix for issue 

* extension controls_if => controls_if_mutator.

* Renamed extension function constant, and moved variables into the mixin.

* Dereference string table references when loading variable fields from JSON.

* Moving FieldImage string dereferencing back into Block.interpolate_() (part of jsonInit()). This sets a clear boundary of where dereferencing should happen.
Towards this, I've added message dereferencing for other field types here, as well.  I've used a pattern of field-type specific helper functions.

* Addressing comments.

* .utils.replaceMessageReferences(..) now gracefully returns non-string arguments.

* Clarification update.
Unraveling nested ternaries in Blockly.utils.tokenizeInterpolation_()

* Code correction from previous commit. Moved style to css.js and set ROUNDING=15;

* Fixing Enter so it properly propogates to dropdown selection. ()

Fixing FieldSegment so it updates dropdowns when the underlying dropdown changes.

* Localisation updates from https://translatewiki.net.

* Make variable add set/get block in context menu obey block limits

* Localisation updates from https://translatewiki.net.

* Use mutator extension for controls_if block

* Fix . Don't check for presence of constants.js

* Fix  (annotations) and an eslint issue (constant condition)

* Localisation updates from https://translatewiki.net.

* Fix : BlockFactory typo and copypasta

* Add safety checks for mutators and non-mutator extensions

* Handle mutations with both mixins and functions

* Adding warning on duplicate JSON block definition.

* period

* Make some functions private and add tests

* Localisation updates from https://translatewiki.net.

* Update help URL per .
Replace URL usages of %28 and %29 with normal parenthesis characters. (They aren't replaced by JavaScript's encodeURIComponent() function, and seem to work just fine without them.)
Added missing semicolon in build.py.

* Typo in comment.

* Make it easier to read the code that creates the variable category in the tolbox

* Adding Blockly.Xml.appendDomToWorkspace() ()

This is a copy (with additional comments) of PR  (and also ) by @qnoirhomme with unrelated files removed. See  for full review.

* Annotation fixes

* Fix bug  by explicitly grabbing focus on the workspace svg element. ()

* Potential fix for .  Stops checking whether we are mid workspace drag since we do not always get mosue up events when blockly is in an iframe. ()

* Adding new minimap demo

* Basic code style changes. Adding a few more comments. Return early if disableScrollChange in onScrollChange listener.

* `unction` to `function` corrects  ()

* Cross browser friendly fix for .  This calls blur and focus from … ()

* Cross browser friendly fix for .  This calls blur and focus from workspace.markFocused and removes the event listener on focus events.  markFocused is called from all of our mouse down handlers, which triggers the focus event leading to an infinite loop of focus.  As far as I can tell, there are no uses of the focus handler that actually did anything for us.

* Localisation updates from https://translatewiki.net.

* Another attempt to fix  to keep the page from jumping to the focused workspace in IE 11 ()

* Adding horizontal scrolling. Changed scroll change callbacks from onScroll_ to setHandlePosition. onScroll_ is not challed when workspace is dragged.

* set background color to lilac if opening the playground from file:

* Registering mousemove and mouseup listener in mousedown event. Mousemove and Mouseup events are now listening over document.

* Localisation updates from https://translatewiki.net.

* Fix  by overriding the updateWidth method in FieldImage blocks to be a no-op. FieldImage fields should not change size after the width is set in init.  The updateWidth and, therefore, getCachedWidth is now being called by BlockSvg renderFields_ (see commit d55d9cbd9f). IIUC, updateWidth/getCachedWidth was only called from render before which is overridden in FieldImage to be a no-op already. ()

* Fix 

* Localisation updates from https://translatewiki.net.

* Fix .  Looks like the original PR just forgot this block. ()

* rebuild develop ()

* Added the variable modal and component and implemented basic renaming functionality. ()

* Fixing commenting from the last commit. ()

* Localisation updates from https://translatewiki.net.

* RemoveAttribute doesn't work on SVG elements in IE 10.  Use setAttribute to null instead.

* Adding the remove variable modal and functionality to accessible Blockly. ()

* Minimap position bug fix for browsers other than chrome. Added touch support.

* Adding an add variable modal to accessible Blockly. ()

* Adding the remove variable modal and functionality to accessible Blockly.

* Adding the add variable modal for accessible Blockly.

* Block browser context menu in the toolbox and flyout

* Add links to the dev registration form and contributor guidelines

* Miscellaneous comment cleanup

* Adding the common modal class. ()

Centralizes accessible modal behavior.

* - Changed error message referencing 'procedure' instead of 'function' ()

- Added iOS specific UI messages
- Fixed bug with js_to_json.py script where it didn't recognize ' character

* - Allows use of Blockly's messaging format for category name, colour,… ()

...in toolbox XML.
- Updated code editor demo to use this message format
- Re-built blockly_compressed.js

* Making text_count use a text color (like text_length, which also returns a number). ()

* Enable google/blockly with continuous build on travis ci () ()

* create .travis for ci job

* initial checkin for blocky-web travis ci job

* rename file to .travis.yaml for typo

* remove after_script

* added cache

* rename .travis.yaml to .travis.yml

* Update .travis.yml

* include build script

* fix yaml file format issue

* debug install part

* debug build issue

* Update .travis.yml

* remove cache for now

* Update .travis.yml

* Update .travis.yml

* Update .travis.yml

* more debug info

* Update .travis.yml

* Update .travis.yml

* fix typo

* installing chrome browser

* remove chrome setting config

* run build.py as part of npm install

* Update .travis.yml

* update karma dependency

* use karma as test runner

* fix typo

* remove karma test for now

* Update .travis.yml

* Update package.json

* add npm test target

* add browserstack-runner depdendency

* update browser support

* fix typo for test target

* fix chrome typo

* added closure dependency

* add google-closure-library

* include blockly_uncompressed.js and core.js dependency

* uncomment out core/*.js files

* add kama job as part of install

* remove browserstack add on for now

* fix karma config typo

* add karma-closure

* add os support

* remove typo config

* include more closure files

* change os back to linux

* use closure-library from node_modules

* change log level back to INFO

* change npm test target to use open browser command instead of karma

* change travis test target to use open command instead of karma

* list current directory

* find what's in current dir

* typo command

* Update .travis.yml

* typo again

* open right index.html

* use right path for index.html

* xdg-open to open default browser on travis

* exit browser after 5s wait

* change timeout to 1 min

* exit after opening up browser

* use browser only

* use karma

* remove un-needed dependency

* clean up script section

* fix typo

* update build status on readme

* initial commit for selenium integration tests

* update selenium jar path

* fix test_runner.js typo

* add more debug info

* check java version

* add && instead of 9288

* fix java path

* add logic to check if selenium is running or not

* add some deugging info

* initial commit to get chromedriver

* add chromedriver flag

* add get_chromedriver.sh to package.json and .travel

* change browser to chrome for now

* fix path issue

* update chromdriver path

* fix path issue again

* more debugging

* add debug msg

* fix typo

* minor fix for getting chromedriver

* install latest chrome browser

* clean up pakcage.json

* use npm target for test run

* remove removing trailing comma

* fix another trailing comma

* updated travis test target

* clean up scripts

* not sure nmp run preinstall

* redirect selenium log to tmp file

* revert writing console log to file

* update test summary

* more clean up

* minor clean up before pull request

* resolved closure-library conflict

1. add closure-library to dependencies instead of devDependencies.
2. add lint back in scripts block

* fix typo (adding comma) in script section

* Renames Blockly.workspaceDragSurface to Blockly.WorkspaceDragSurface.

Fixes .

* Ensure useDragSurface is a boolean.

Fixed 

* use pretest instead of preinstall in package.json ()

* cherry pick for pretest fix

* put pretest target to test_setup.sh

* fix conflict

* cherry pick for get_chromedriver.sh

* add some sleep to wait download to finish

* use node.js stable

* use npm test target

* field_angle renders degree symbol consistently.

Fixes 

* bumpNeighbours_ function moved to block_svg.

Fixed 

* Update RegEx in js-to-json to match windowi eol ()

The current regex only works with the "\n" line endings as it expects no characters after the optional ";" at the end of the line. In windows, if it adds the "\r" it counts as a characters and is not part of the line terminator so it doesn't match.

* Fix French translation of "colour with rgb" block ()

"colorier", which is currently used, is a verb and proposed "couleur" is
a noun: the block in question does not change colour of anything, it
creates new colour instead, thus noun is more applicable.

Also, noun is used in French translation of "random colour" block:
"couleur aléatoire".

* Enforcing non-empty names on value inputs and statement inputs. ()

* Correcting  ()

single quotes. better logic.

* Created a variable model with name, id, and type.

Created a jsunit test file for variable model.

* Change how blockly handles cursors.  The old way was quite slow becau… ()

* Change how blockly handles cursors.  The old way was quite slow because it changed the stylesheet directly.  See issue  for more details on implementation and tradeoffs.  This changes makes the following high level changes: deprecate Blockly.Css.setCursor, use built in open and closed hand cursor instead of custom .cur files, add css to draggable objects to set the open and closed hand cursors.

* Rebuild blockly_uncompressed to pick up a testing change to make travis happy.  Fix a build warning from a multi-line string in the process. ()

* Merge master into develop ()

- pick up translation changes
- clean up trailing spaces

* use goog.string.startswith instead of string.startswith ()

* New jsinterpreter demo includes wait block. Both demos have improved UI for clarity. ()

Refactor of interpreter demo
 * Renamed demos/interpreter/index.html as demos/interpreter/step-execution.html (including redirect), and added demos/interpreter/async-execution.html.
 * Refactored code to automatically generate/parse the blocks, eliminating the need for a "Parse JavaScript" button. Code is still shown in alert upon stepping to the first statement. Print statements now write to output <textarea> instead of modal dialogs.

* VariableMap and functions added.

* Create separate file for VariableMap and its functions.

* Fix  ()

* VariableMap and functions added.

* Fix  ()

* Improve errors when validating JSON block definitions. ()

goog.asserts to not run from blockly_compressed.js. User data validation should always run.

* Dragging changes, rebased on develop ()

* Add block drag surface translateSurfaceBy

* Add dragged connection manager

* Add gesture.js

* Add GestureHandler

* Implemented gesture skeleton

* Most basic workspace dragging

* Add dragged connection manager

* cleanup

* doc

* more cleanup

* Add gesture handler

* Add translateSurfaceInternal

* core/block_dragger.js

* cleanup

* Pull in changes to dragged connection manager

* Pull in changes to dragged connection manager

* comments

* more annotations

* Add workspace dragger

* Add coordinate annotations

* Start on block dragging

* Limit number of concurrent gestures

* Add some TODOs

* start using dragged connection manager

* Set origin correctly for dragging blocks

* Connect or delete at the end of a block drag.

* cleanup

* handle field clicks and block + workspace right-clicks

* move code into BlockDragger class, but still reach into Gesture internals a lot

* Clean up block dragger

* Call blockDragger constructor with correct arguments

* Enable block dragging in a mutator workspace

* Add workspace dragger

* click todos

* Drag flyout with background

* more dragging from flyout

* nit

* fix dragging from flyouts

* Remove unused code and rename gestureHandler to gestureDB

* Rename gesture handler

* Added some jsdoc in gesture.js

* Update some docs

* Move some code to block_svg and clean up code

* Lots of coordinate annotations

* Fix block dragging when zoomed.

* Remove built files from branch

* More dragging work ()

-- Drag bubbles while dragging blocks
-- Use bindEventWithChecks to work in touch on Android. Not tested anywhere else yet.
-- Handle dragging blocks while zoomed
-- Handle dragging blocks in mutators
-- Handle right-clicks (I hope)
-- Removed lots of unused code

* More dragging work ()

- Removed gestureDB
- Removing uses of terminateDrag
- Cleaned up disposal code

* Dragging bugfixes ()

- Get rid of flyout.dragMode_ and blockly.dragMode_
- Make drags from the flyout start from the top block in the group
- Block tooltips from being scheduled or shown during gestures
- Don't resize mutator bubbles mid-drag

* Fix events in new dragging ()

* rebuild for testing

* unbuild

* Fix events

* rebuild

* Fix up cursors

* Use language files from develop

* Remove handled TODOS

* attempt to fix IE rerendering bug, and recalculate workspace positions on scroll

* Rebuild all the things

* Comment cleanup; annotations; delete unused variables.

* Tidy up context menu code. ()

* add osx travis test run job ()

* Names are correctly fetched from VariableModels!

* add more wait time for test setup ()

* Work around timing issue with travis osx issue ()

* add more wait time for test setup

* increase selenium wait time

* add more wait

* Fix  by adding a rule to cover the toolbox labels too. ()

* Assign variable UUID to field_variable dropdown.

* Change registration link to a static one ()

This lets us redirect to a different form if we change it in the future.

* Edit generators to read in Variable Models.

* Add VariableMap requirement to workspace.

* Changed parameter name in workspace for clarity.

* Add type, id, and info to the generated xml.

Add xml tests for fieldToDom.
Update workspace tests to pass with new changes.

* Fix apostrophe in tooltips and helpurls ()

* Click events on shadow blocks have the correct id ()

* Add image_onclick option ()

* Cleanup: semicolons, spacing, etc. ()

* Spelling.  Spelling is hard.

* Add variable info to xml generated in variables.js

* Add missing CLAs info to the contributing file ()

* Add missing CLAs info to the contributing file

* Added larger changes paragraph

* Replacing latest prettifier hosted in repo with latest version at rawgit CDN. ()

* Forgot update code demo ()

* Move audio code to a new file ()

* move audio code to a new file

* dispose

* null check

* Make flyout get variables from target workspace's variableMap.

* Require VariableModel in field_variable.js.

* Update contributing.md ()

* Include variables at top of serialization.

* Move blockSvg.getHeightWidth to block_render_svg.js ()

* Deserialization variables at top.

* Create grid object ()

* Create grid object

* Doc

* Units!  Thanks @RoboErikG

* Add our sound files in other formats

* Remove duplicate merged code

* Remove translation files

* Cleanup duplicate functions

* Add back scrollbar code

* Fix variable related issues

* Revert workspace variable creation change

* Delete language files

* Appease linter

* Remove duplication

* Add spaces to jsdoc line
2017-05-30 09:26:06 -04:00
Rachel Fenichel
6275e1137c New dragging! Merge from google ()
Port of a major refactor from Blockly.  Dragging logic now lives in block_dragger.js, gesture.js, workspace_dragger.js, dragged_connection_manager.js (unused by scratch-blocks), and insertion_marker_manager.js (used only by scratch-blocks).
2017-05-22 13:08:22 -07:00
Paul Kaplan
1827255261 Merge google/blockly, May 2017 ()
* Move createDom call into the constructor of block drag surface. ()

* Make cursor stay as a closed hand when dragging blocks around in the drag surface. Do this by applying the same style to text elements in the drag surface that we do in the main svg. ()

* Don't connect to blocks under the flyout.

* recompile again. ()

* Fix german translation

* Option for moving one block from stack.

See thread in support group before merging.

* Fix german translation of 'delete x blocks'

* Adding unit tests for ifelse block.

* Improvements to the generator test framework.

* <field>, <value> reorder due to load/save.

* Expand stack-drag modifier key to include alt and ctrl.

* Use the npm closure library instead of the same library installed at a parallel directory

* Fix undo/redo for FieldCheckbox

Thanks to PR  by ademenev

* Localisation updates from https://translatewiki.net.

* PR : Adding support for string table lookups in dropdown field labels

Adding support for string table lookups in dropdown field labels specified in JSON.

Adds Blockly.utils.replaceMessageReferences() method to handle string replacement without interpolation tokens.  Effectively uses the same old code, now moved into tokenizeInterpolation_(), which takes a parseInterpolationTokens option.

Replaces the direct JavaScript references (not pure JSON, and thus not portable).

Demonstrating this behavior in the logic_boolean dropdown.

* Integrating qqq.json changes into messages.json. ()

From commits b77f8cbebc and 4ecdedec9f

* Naming changes in mirror demo

* Adding support for untranslated messages. ()

This will be used to define constants accessible in JSON block definitions. Messages with descriptions that include `{{Notranslate}}` will not be included in the translation files sent to TranslateWiki. Instead, they are written to `msg/json/constants.json`, and later merged back into the `.js` files, similar to synonyms.

Template details: https://translatewiki.net/wiki/Template:Notranslate

* JSON support for message lookup in colour, tooltip, and help URL. ()

String replacement for the colour, tooltip text and help URL attributes of JSON defined blocks.
Demonstrated in logic_boolean.

* Fixes as per code review on PR.

* Localisation updates from https://translatewiki.net.

* Reduce number of Closure files in App Engine upload.

* Python false is False.  Issue .

* Replace 'const' with 'var'.

This unbreaks IE10 and advanced compiled apps such as Blockly Games.

* Fix bug in audioService where attached event callbacks were not being cleared properly.

* Rename workspace-tree to workspace-block.

* Minor refactoring of the modal code (add comments, guard against invalid keystrokes, etc.).

* FieldNumber & FieldAngle: Default value "0" ()

FieldNumber and FieldAngle previously accepted "undefined" as values, if not defined in JSON. This catches these and uses "0" for any NaN value.  The constructor value parameter is now optional. Includes tests.

* Remove unnecessary check when attaching a new block to a marked connection.

* Remove debug info.

* Refactor and simplify field-segment.component.js.

* Replace single quotes with double. ()

Fixes commits in .

* Adding extensions for JSON support of dynamic blocks. ()

Adding support for extensions, functions that can assist with loading blocks, much like init functions, but that can be referenced from JSON definitions. This allows JSON definitions to define dynamic blocks such as onchange handlers and mutators.

Rewrote math_number as an example pure JSON block.

* Add ability to add a class to a scrollbar so that different types of … ()

* Add ability to add a class to a scrollbar so that different types of scrollbars can
be distinguished from each other. You used to be able to do this by looking at the parent
element but now all the scrollbars are siblings in the dom.

Also, use this new class to fix  so that layering of the flyout and workspace scrollbars
are done correctly.

* JSON definitions for colour blocks ()

Replaces old colour block definitions with a Blockly.defineBlocksWithJsonArray(..) call. Generator unit tests continue to load and pass, signifying compatibility with prior block definitions.

Replaces extension 'math_number_tooltip' with the reusable 'parent_tooltip_when_inline' extension, also used by colour_picker. Includes tests.

* Rewrite tree.service.js.

- Remove unnecessary code and functions.
- Add documentation where needed.
- Fix a bug arising when a block on the workspace is attached to an existing link.

* Use setValue in fieldTextInput so that procedure renaming works

* Further cleanup and removal of unnecessary functions. Pull some strings out for i18n.

* Use bindEvent_ instead of bindEventWithChecks_ for longStop

* Clean up workspace.component.js. When moving a block from one place to another, move all blocks after it too, and adjust the active descs accordingly.

* Unit tests for JSON block definitions (just the start) ()

 * Beginnings of a JSON block definition unit test set.
 * Dispose of unit test workspaces and blocks in finally blocks.
 * Clarify JSON error message by echoing arg notation.

* New blocks text_count, text_replace, and text_reverse ()

Includes generators for all languages and units tests on those generators.

* Fixing combo boxes getting out-of-sync with NVDA.

Combo boxes need to be special cased like text input. Also, Escape is
a reserved button in NVDA, so I added Enter as a way to "submit and
move up a level" in addition to escape, so these boxes can be edited
while NVDA is on.

* Temporary fix for broken text field validation.

* rebuild

* Add a block to reverse a list ()

* Localisation updates from https://translatewiki.net.

* Porting math.js blocks to JSON ()

Moving all `math.js` definitions into a single JSON array, complete with i18n syntax for all messages, dropdowns, and tooltips.

Adding Blockly.Extensions.buildTooltipForDropdown(..) to facilitate the creation and error-checking of tooltips that update based on the value of a dropdown.

Now warn on raw string in JSON 'extensions'.

* Fixing JSON support for images in dropdowns. Adding tests. ()

Fixes .

* Update README.md

Add a link to our forum.

* Correcting math_change color

* Enable custom flyout categories.

* Add some safety

* Update the set of reserved words in Python to reflect the current state of Python (2.7 and 3.6). ()

* Localisation updates from https://translatewiki.net.

* .getOptions_() to .getOptions() ()

Fixes .

* Blockly.Extensions.buildTooltipForDropdown(..): Deferred validation. ()

Defer tooltip message string check until after load, when all Blockly.Msg should be loaded.
Avoids validation in headless mode, due to lack of document.readyState.

* annotation updates

* annotation updates

* jsdoc corrections ()

* Remove use of Array.prototype.includes which is not implemented in IE or Edge < 14. Fixes .

* Attempt to work around the IE/Edge bug where `getComputedTextLength()` throws an exception when the SVG node is not visible. This workaround forces a re-render, which in turn, forces a re-calculation of the node width once a block is inserted into the workspace SVG. This workaround is only executed on IE and Edge. See https://groups.google.com/forum/#!topic/blockly/T8IR4t4xAIY for the initial discussion of this issue.

* Change CSS transforms to work with older browsers ()

* Change the setting of the CSS transform properties on SVG nodes to set both the unprefixed version and the `-webkit-` prefixed version so that Blockly correctly renders in order browsers, such as Safari < 9 and iOS Safari < 9.2. For discussion of this issue, see https://groups.google.com/forum/#!topic/blockly/o3pERaRQhSg

* Correct the separation between the CSS transform property and the rest of the CSS that was in the variable misleadingly called "transform".

* Don't try to get block position in a headless workspace

* Stop bumping neighbours in headless blockly

* Place context menu correctly on touch

* Clear all active desc ids when the 'Erase Workspace' button is pressed.

* Fix a bug where splicing a block between two linked blocks disconnects the group and messes up the focus.

* Deleting a top-level block does not cause blocks after it to be deleted. Properly handle the active desc for this case.

* Localisation updates from https://translatewiki.net.

* Use the empty field placeholder for dropdowns that do not have a value selected.

* Bugfix for . I incorrectly converted one CSS transform setting to use the cross-browser setting function in 40a063763c74b3f712c3057565966c25d5cfdb10. ()

* Adding @namespace annotations for JSDoc. ()

* Fix typo causing TypeError ()

* Pinning the angular2 dependency, and including licenses. ()

* Add skeleton for tests on rendered workspaces

* Fix some lint errors

* Localisation updates from https://translatewiki.net.

* Correct changedState in setWarningText() ()

When clearing warnings on blocks with IDs, the changedState variable should be true if the text changed. This will trigger the block being reshaped and remove the space for the notification icon (this.bumpNeighbours_).

* Adds Block.prototype.mixin() and Blockly.Extensions.registerMixin(). ()

Adds Block.prototype.mixin() and Blockly.Extensions.registerMixin().
This adds support for a common use pattern in extensions, and adds
error checking to avoid future incompatibilities.

* Porting Logic blocks to JSON ()

Extensions, mixins, mutators and constants now grouped under the new namespace Blockly.Constants.Logic.

* Improving errors/warnings with Block.toDevString() and Connection.toString(). ()

* Add isEditable to field, and add tests

* Separate tests

* Blockly.Constants.Math and Blockly.Constants.Colour extension constants ()

Also, correcting quotes in logic.js.

* Correction to logic_ternary type check ()

* Porting Loop blocks to JSON ()

 * Improved documentation on `Blockly.Extensions.buildTooltipForDropdown`
 * Replaced incorrect uses of `@mixes` JSDoc annotation (on mixin extensions) with `@augments Blockly.Block`.
 * Added Blockly.Extensions.buildTooltipWithFieldValue() extension helper.

* Workspace isDraggable

* JSONify simple list blocks

* JSONify variable blocks

* Initial text block, with a mixin to generate quote image fields. ()

Text block now uses the extension "text_quotes", supported by Blockly.Constants.Text.QUOTE_IMAGE_MIXIN.quoteField_(fieldName), so that each platform can use the best platform appropriate image (size, density, etc.) for the quotes.

* Add no-op stub .neighbors() for headless Connection.

* Adding tests for logic_ternary block in a new jsunit test framework.

* Correcting output of the logic_null block.

* Potential bug fix for issue 

* extension controls_if => controls_if_mutator.

* Renamed extension function constant, and moved variables into the mixin.

* Dereference string table references when loading variable fields from JSON.

* Moving FieldImage string dereferencing back into Block.interpolate_() (part of jsonInit()). This sets a clear boundary of where dereferencing should happen.
Towards this, I've added message dereferencing for other field types here, as well.  I've used a pattern of field-type specific helper functions.

* Addressing comments.

* .utils.replaceMessageReferences(..) now gracefully returns non-string arguments.

* Clarification update.
Unraveling nested ternaries in Blockly.utils.tokenizeInterpolation_()

* Code correction from previous commit. Moved style to css.js and set ROUNDING=15;

* Fixing Enter so it properly propogates to dropdown selection. ()

Fixing FieldSegment so it updates dropdowns when the underlying dropdown changes.

* Localisation updates from https://translatewiki.net.

* Make variable add set/get block in context menu obey block limits

* Localisation updates from https://translatewiki.net.

* Use mutator extension for controls_if block

* Fix . Don't check for presence of constants.js

* Fix  (annotations) and an eslint issue (constant condition)

* Localisation updates from https://translatewiki.net.

* Fix : BlockFactory typo and copypasta

* Add safety checks for mutators and non-mutator extensions

* Handle mutations with both mixins and functions

* Adding warning on duplicate JSON block definition.

* period

* Make some functions private and add tests

* Localisation updates from https://translatewiki.net.

* Update help URL per .
Replace URL usages of %28 and %29 with normal parenthesis characters. (They aren't replaced by JavaScript's encodeURIComponent() function, and seem to work just fine without them.)
Added missing semicolon in build.py.

* Typo in comment.

* Make it easier to read the code that creates the variable category in the tolbox

* Adding Blockly.Xml.appendDomToWorkspace() ()

This is a copy (with additional comments) of PR  (and also ) by @qnoirhomme with unrelated files removed. See  for full review.

* Annotation fixes

* Fix bug  by explicitly grabbing focus on the workspace svg element. ()

* Potential fix for .  Stops checking whether we are mid workspace drag since we do not always get mosue up events when blockly is in an iframe. ()

* Adding new minimap demo

* Basic code style changes. Adding a few more comments. Return early if disableScrollChange in onScrollChange listener.

* `unction` to `function` corrects  ()

* Cross browser friendly fix for .  This calls blur and focus from … ()

* Cross browser friendly fix for .  This calls blur and focus from workspace.markFocused and removes the event listener on focus events.  markFocused is called from all of our mouse down handlers, which triggers the focus event leading to an infinite loop of focus.  As far as I can tell, there are no uses of the focus handler that actually did anything for us.

* Localisation updates from https://translatewiki.net.

* Another attempt to fix  to keep the page from jumping to the focused workspace in IE 11 ()

* Adding horizontal scrolling. Changed scroll change callbacks from onScroll_ to setHandlePosition. onScroll_ is not challed when workspace is dragged.

* set background color to lilac if opening the playground from file:

* Registering mousemove and mouseup listener in mousedown event. Mousemove and Mouseup events are now listening over document.

* Localisation updates from https://translatewiki.net.

* Fix  by overriding the updateWidth method in FieldImage blocks to be a no-op. FieldImage fields should not change size after the width is set in init.  The updateWidth and, therefore, getCachedWidth is now being called by BlockSvg renderFields_ (see commit d55d9cbd9f). IIUC, updateWidth/getCachedWidth was only called from render before which is overridden in FieldImage to be a no-op already. ()

* Fix 

* Localisation updates from https://translatewiki.net.

* Fix .  Looks like the original PR just forgot this block. ()

* rebuild develop ()

* Added the variable modal and component and implemented basic renaming functionality. ()

* Fixing commenting from the last commit. ()

* Localisation updates from https://translatewiki.net.

* RemoveAttribute doesn't work on SVG elements in IE 10.  Use setAttribute to null instead.

* Adding the remove variable modal and functionality to accessible Blockly. ()

* Minimap position bug fix for browsers other than chrome. Added touch support.

* Adding an add variable modal to accessible Blockly. ()

* Adding the remove variable modal and functionality to accessible Blockly.

* Adding the add variable modal for accessible Blockly.

* Block browser context menu in the toolbox and flyout

* Add links to the dev registration form and contributor guidelines

* Miscellaneous comment cleanup

* Adding the common modal class. ()

Centralizes accessible modal behavior.

* - Changed error message referencing 'procedure' instead of 'function' ()

- Added iOS specific UI messages
- Fixed bug with js_to_json.py script where it didn't recognize ' character

* - Allows use of Blockly's messaging format for category name, colour,… ()

...in toolbox XML.
- Updated code editor demo to use this message format
- Re-built blockly_compressed.js

* Making text_count use a text color (like text_length, which also returns a number). ()

* Enable google/blockly with continuous build on travis ci () ()

* create .travis for ci job

* initial checkin for blocky-web travis ci job

* rename file to .travis.yaml for typo

* remove after_script

* added cache

* rename .travis.yaml to .travis.yml

* Update .travis.yml

* include build script

* fix yaml file format issue

* debug install part

* debug build issue

* Update .travis.yml

* remove cache for now

* Update .travis.yml

* Update .travis.yml

* Update .travis.yml

* more debug info

* Update .travis.yml

* Update .travis.yml

* fix typo

* installing chrome browser

* remove chrome setting config

* run build.py as part of npm install

* Update .travis.yml

* update karma dependency

* use karma as test runner

* fix typo

* remove karma test for now

* Update .travis.yml

* Update package.json

* add npm test target

* add browserstack-runner depdendency

* update browser support

* fix typo for test target

* fix chrome typo

* added closure dependency

* add google-closure-library

* include blockly_uncompressed.js and core.js dependency

* uncomment out core/*.js files

* add kama job as part of install

* remove browserstack add on for now

* fix karma config typo

* add karma-closure

* add os support

* remove typo config

* include more closure files

* change os back to linux

* use closure-library from node_modules

* change log level back to INFO

* change npm test target to use open browser command instead of karma

* change travis test target to use open command instead of karma

* list current directory

* find what's in current dir

* typo command

* Update .travis.yml

* typo again

* open right index.html

* use right path for index.html

* xdg-open to open default browser on travis

* exit browser after 5s wait

* change timeout to 1 min

* exit after opening up browser

* use browser only

* use karma

* remove un-needed dependency

* clean up script section

* fix typo

* update build status on readme

* initial commit for selenium integration tests

* update selenium jar path

* fix test_runner.js typo

* add more debug info

* check java version

* add && instead of 9288

* fix java path

* add logic to check if selenium is running or not

* add some deugging info

* initial commit to get chromedriver

* add chromedriver flag

* add get_chromedriver.sh to package.json and .travel

* change browser to chrome for now

* fix path issue

* update chromdriver path

* fix path issue again

* more debugging

* add debug msg

* fix typo

* minor fix for getting chromedriver

* install latest chrome browser

* clean up pakcage.json

* use npm target for test run

* remove removing trailing comma

* fix another trailing comma

* updated travis test target

* clean up scripts

* not sure nmp run preinstall

* redirect selenium log to tmp file

* revert writing console log to file

* update test summary

* more clean up

* minor clean up before pull request

* resolved closure-library conflict

1. add closure-library to dependencies instead of devDependencies.
2. add lint back in scripts block

* fix typo (adding comma) in script section

* Renames Blockly.workspaceDragSurface to Blockly.WorkspaceDragSurface.

Fixes .

* Ensure useDragSurface is a boolean.

Fixed 

* use pretest instead of preinstall in package.json ()

* cherry pick for pretest fix

* put pretest target to test_setup.sh

* fix conflict

* cherry pick for get_chromedriver.sh

* add some sleep to wait download to finish

* use node.js stable

* use npm test target

* field_angle renders degree symbol consistently.

Fixes 

* bumpNeighbours_ function moved to block_svg.

Fixed 

* Update RegEx in js-to-json to match windowi eol ()

The current regex only works with the "\n" line endings as it expects no characters after the optional ";" at the end of the line. In windows, if it adds the "\r" it counts as a characters and is not part of the line terminator so it doesn't match.

* Fix French translation of "colour with rgb" block ()

"colorier", which is currently used, is a verb and proposed "couleur" is
a noun: the block in question does not change colour of anything, it
creates new colour instead, thus noun is more applicable.

Also, noun is used in French translation of "random colour" block:
"couleur aléatoire".

* Enforcing non-empty names on value inputs and statement inputs. ()

* Correcting  ()

single quotes. better logic.

* Created a variable model with name, id, and type.

Created a jsunit test file for variable model.

* Change how blockly handles cursors.  The old way was quite slow becau… ()

* Change how blockly handles cursors.  The old way was quite slow because it changed the stylesheet directly.  See issue  for more details on implementation and tradeoffs.  This changes makes the following high level changes: deprecate Blockly.Css.setCursor, use built in open and closed hand cursor instead of custom .cur files, add css to draggable objects to set the open and closed hand cursors.

* Rebuild blockly_uncompressed to pick up a testing change to make travis happy.  Fix a build warning from a multi-line string in the process. ()

* Merge master into develop ()

- pick up translation changes
- clean up trailing spaces

* use goog.string.startswith instead of string.startswith ()

* New jsinterpreter demo includes wait block. Both demos have improved UI for clarity. ()

Refactor of interpreter demo
 * Renamed demos/interpreter/index.html as demos/interpreter/step-execution.html (including redirect), and added demos/interpreter/async-execution.html.
 * Refactored code to automatically generate/parse the blocks, eliminating the need for a "Parse JavaScript" button. Code is still shown in alert upon stepping to the first statement. Print statements now write to output <textarea> instead of modal dialogs.

* Fix  ()

* Fix cursor and mistaken css from merge

* Comment out broken field angle merge

* Fix broken merge with borders

* Add back original package json (woops)

* Remove render function for field angle, may not be the right way but it was broken...

* Revert merge blocking variable shadow blocks

* Add changes from built lang files

* Add back travis and readme

* Revert broken cleanup additions

* Add notes for scratch-block specific functions

* Revert change to css so blocks stay under the toolbox

* Add back accidentally removed files

* Use getFlyout_ instead of getFlyout everywhere

* Satisfy the linter

* Re-remove deprecated function

* Remove duplicated code in block_svg

* Add back flip_rtl option for images

* Remove more duplicated functions from past merges

* Fix flip_rtl code

* Revert renaming of getFlyout
2017-05-11 15:58:18 -04:00