Fix up annotations

This commit is contained in:
Rachel Fenichel
2019-08-23 14:52:42 -07:00
parent a57f4691c7
commit a9189f01bc
4 changed files with 28 additions and 30 deletions

View File

@@ -95,7 +95,7 @@ goog.addDependency("../../../" + dir + "/core/rendered_connection.js", ['Blockly
goog.addDependency("../../../" + dir + "/core/renderers/block_rendering_rewrite/block_render_draw.js", ['Blockly.blockRendering.Drawer'], ['Blockly.blockRendering.Debug', 'Blockly.blockRendering.RenderInfo', 'Blockly.blockRendering.Measurable', 'Blockly.blockRendering.BottomRow', 'Blockly.blockRendering.InputRow', 'Blockly.blockRendering.Row', 'Blockly.blockRendering.SpacerRow', 'Blockly.blockRendering.TopRow']);
goog.addDependency("../../../" + dir + "/core/renderers/block_rendering_rewrite/block_render_draw_debug.js", ['Blockly.blockRendering.Debug'], ['Blockly.blockRendering.RenderInfo', 'Blockly.blockRendering.Measurable', 'Blockly.blockRendering.BottomRow', 'Blockly.blockRendering.InputRow', 'Blockly.blockRendering.Row', 'Blockly.blockRendering.SpacerRow', 'Blockly.blockRendering.TopRow']);
goog.addDependency("../../../" + dir + "/core/renderers/block_rendering_rewrite/block_render_info.js", ['Blockly.blockRendering.RenderInfo'], ['Blockly.blockRendering.Measurable', 'Blockly.blockRendering.BottomRow', 'Blockly.blockRendering.InputRow', 'Blockly.blockRendering.Row', 'Blockly.blockRendering.SpacerRow', 'Blockly.blockRendering.TopRow', 'Blockly.blockRendering.InlineInput', 'Blockly.blockRendering.ExternalValueInput', 'Blockly.blockRendering.StatementInput', 'Blockly.blockRendering.PreviousConnection', 'Blockly.blockRendering.NextConnection', 'Blockly.blockRendering.OutputConnection', 'Blockly.RenderedConnection']);
goog.addDependency("../../../" + dir + "/core/renderers/block_rendering_rewrite/block_rendering.js", ['Blockly.blockRendering'], ['Blockly.blockRendering.Debug', 'Blockly.blockRendering.Drawer', 'Blockly.blockRendering.RenderInfo', 'Blockly.blockRendering.ConstantProvider', 'Blockly.geras.HighlightConstantProvider', 'Blockly.geras.RenderInfo', 'Blockly.thrasos.RenderInfo', 'Blockly.zelos.RenderInfo']);
goog.addDependency("../../../" + dir + "/core/renderers/block_rendering_rewrite/block_rendering.js", ['Blockly.blockRendering'], ['Blockly.blockRendering.Debug', 'Blockly.blockRendering.Drawer', 'Blockly.blockRendering.RenderInfo', 'Blockly.blockRendering.ConstantProvider', 'Blockly.geras.HighlightConstantProvider', 'Blockly.geras.RenderInfo', 'Blockly.geras.Drawer', 'Blockly.thrasos.RenderInfo', 'Blockly.zelos.RenderInfo']);
goog.addDependency("../../../" + dir + "/core/renderers/block_rendering_rewrite/block_rendering_constants.js", ['Blockly.blockRendering.ConstantProvider'], ['Blockly.utils.svgPaths']);
goog.addDependency("../../../" + dir + "/core/renderers/block_rendering_rewrite/measurables.js", ['Blockly.blockRendering.InRowSpacer'], ['Blockly.blockRendering.Measurable']);
goog.addDependency("../../../" + dir + "/core/renderers/geras/drawer.js", ['Blockly.geras.Drawer'], ['Blockly.blockRendering.ConstantProvider', 'Blockly.blockRendering.Drawer', 'Blockly.geras.Highlighter', 'Blockly.geras.RenderInfo']);

View File

@@ -62,9 +62,9 @@ Blockly.blockRendering.Drawer = function(block, info) {
* joined with spaces and set directly on the block. This guarantees that
* the steps are separated by spaces for improved readability, but isn't
* required.
* @private
* @package
*/
Blockly.blockRendering.Drawer.prototype.draw_ = function() {
Blockly.blockRendering.Drawer.prototype.draw = function() {
this.hideHiddenIcons_();
this.drawOutline_();
this.drawInternals_();
@@ -82,7 +82,7 @@ Blockly.blockRendering.Drawer.prototype.draw_ = function() {
* Save sizing information back to the block
* Most of the rendering information can be thrown away at the end of the render.
* Anything that needs to be kept around should be set in this function.
* @private
* @protected
*/
Blockly.blockRendering.Drawer.prototype.recordSizeOnBlock_ = function() {
// This is used when the block is reporting its size to anyone else.
@@ -95,7 +95,7 @@ Blockly.blockRendering.Drawer.prototype.recordSizeOnBlock_ = function() {
/**
* Hide icons that were marked as hidden.
* @private
* @protected
*/
Blockly.blockRendering.Drawer.prototype.hideHiddenIcons_ = function() {
for (var i = 0, iconInfo; (iconInfo = this.info_.hiddenIcons[i]); i++) {
@@ -105,7 +105,7 @@ Blockly.blockRendering.Drawer.prototype.hideHiddenIcons_ = function() {
/**
* Create the outline of the block. This is a single continuous path.
* @private
* @protected
*/
Blockly.blockRendering.Drawer.prototype.drawOutline_ = function() {
this.drawTop_();
@@ -129,7 +129,7 @@ Blockly.blockRendering.Drawer.prototype.drawOutline_ = function() {
/**
* Add steps for the top corner of the block, taking into account
* details such as hats and rounded corners.
* @private
* @protected
*/
Blockly.blockRendering.Drawer.prototype.drawTop_ = function() {
var topRow = this.info_.topRow;
@@ -157,7 +157,7 @@ Blockly.blockRendering.Drawer.prototype.drawTop_ = function() {
/**
* Add steps for the jagged edge of a row on a collapsed block.
* @param {!Blockly.blockRendering.Row} row The row to draw the side of.
* @private
* @protected
*/
Blockly.blockRendering.Drawer.prototype.drawJaggedEdge_ = function(row) {
var remainder =
@@ -171,7 +171,7 @@ Blockly.blockRendering.Drawer.prototype.drawJaggedEdge_ = function(row) {
* of the block.
* @param {!Blockly.blockRendering.Row} row The row that this input
* belongs to.
* @private
* @protected
*/
Blockly.blockRendering.Drawer.prototype.drawValueInput_ = function(row) {
var input = row.getLastInput();
@@ -188,7 +188,7 @@ Blockly.blockRendering.Drawer.prototype.drawValueInput_ = function(row) {
* Add steps for a statement input.
* @param {!Blockly.blockRendering.Row} row The row that this input
* belongs to.
* @private
* @protected
*/
Blockly.blockRendering.Drawer.prototype.drawStatementInput_ = function(row) {
var input = row.getLastInput();
@@ -217,7 +217,7 @@ Blockly.blockRendering.Drawer.prototype.drawStatementInput_ = function(row) {
* statement input connections.
* @param {!Blockly.blockRendering.Row} row The row to draw the
* side of.
* @private
* @protected
*/
Blockly.blockRendering.Drawer.prototype.drawRightSideRow_ = function(row) {
this.outlinePath_ +=
@@ -229,7 +229,7 @@ Blockly.blockRendering.Drawer.prototype.drawRightSideRow_ = function(row) {
/**
* Add steps for the bottom edge of a block, possibly including a notch
* for the next connection
* @private
* @protected
*/
Blockly.blockRendering.Drawer.prototype.drawBottom_ = function() {
var bottomRow = this.info_.bottomRow;
@@ -255,7 +255,7 @@ Blockly.blockRendering.Drawer.prototype.drawBottom_ = function() {
/**
* Add steps for the left side of the block, which may include an output
* connection
* @private
* @protected
*/
Blockly.blockRendering.Drawer.prototype.drawLeft_ = function() {
var outputConnection = this.info_.outputConnection;
@@ -277,7 +277,7 @@ Blockly.blockRendering.Drawer.prototype.drawLeft_ = function() {
/**
* Draw the internals of the block: inline inputs, fields, and icons. These do
* not depend on the outer path for placement.
* @private
* @protected
*/
Blockly.blockRendering.Drawer.prototype.drawInternals_ = function() {
for (var i = 0, row; (row = this.info_.rows[i]); i++) {
@@ -295,7 +295,7 @@ Blockly.blockRendering.Drawer.prototype.drawInternals_ = function() {
* Push a field or icon's new position to its SVG root.
* @param {!Blockly.blockRendering.Icon|!Blockly.blockRendering.Field} fieldInfo
* The rendering information for the field or icon.
* @private
* @protected
*/
Blockly.blockRendering.Drawer.prototype.layoutField_ = function(fieldInfo) {
if (fieldInfo.isField()) {
@@ -334,7 +334,7 @@ Blockly.blockRendering.Drawer.prototype.layoutField_ = function(fieldInfo) {
* Add steps for an inline input.
* @param {Blockly.blockRendering.InlineInput} input The information about the
* input to render.
* @private
* @protected
*/
Blockly.blockRendering.Drawer.prototype.drawInlineInput_ = function(input) {
var width = input.width;
@@ -362,7 +362,7 @@ Blockly.blockRendering.Drawer.prototype.drawInlineInput_ = function(input) {
* parent block's dark path show through.
* @param {Blockly.blockRendering.InlineInput} input The information about
* the input that the connection is on.
* @private
* @protected
*/
Blockly.blockRendering.Drawer.prototype.positionInlineInputConnection_ = function(input) {
var yPos = input.centerline - input.height / 2;
@@ -385,7 +385,7 @@ Blockly.blockRendering.Drawer.prototype.positionInlineInputConnection_ = functio
* RTL and the small gap between the parent block and child block which lets the
* parent block's dark path show through.
* @param {!Blockly.blockRendering.Row} row The row that the connection is on.
* @private
* @protected
*/
Blockly.blockRendering.Drawer.prototype.positionStatementInputConnection_ = function(row) {
var input = row.getLastInput();
@@ -405,7 +405,7 @@ Blockly.blockRendering.Drawer.prototype.positionStatementInputConnection_ = func
* RTL and the small gap between the parent block and child block which lets the
* parent block's dark path show through.
* @param {!Blockly.blockRendering.Row} row The row that the connection is on.
* @private
* @protected
*/
Blockly.blockRendering.Drawer.prototype.positionExternalValueConnection_ = function(row) {
var input = row.getLastInput();
@@ -421,7 +421,7 @@ Blockly.blockRendering.Drawer.prototype.positionExternalValueConnection_ = funct
/**
* Position the previous connection on a block.
* @private
* @protected
*/
Blockly.blockRendering.Drawer.prototype.positionPreviousConnection_ = function() {
var topRow = this.info_.topRow;
@@ -434,7 +434,7 @@ Blockly.blockRendering.Drawer.prototype.positionPreviousConnection_ = function()
/**
* Position the next connection on a block.
* @private
* @protected
*/
Blockly.blockRendering.Drawer.prototype.positionNextConnection_ = function() {
var bottomRow = this.info_.bottomRow;
@@ -451,7 +451,7 @@ Blockly.blockRendering.Drawer.prototype.positionNextConnection_ = function() {
/**
* Position the output connection on a block.
* @private
* @protected
*/
Blockly.blockRendering.Drawer.prototype.positionOutputConnection_ = function() {
if (this.info_.outputConnection) {

View File

@@ -37,6 +37,7 @@ goog.require('Blockly.blockRendering.ConstantProvider');
goog.require('Blockly.geras.HighlightConstantProvider');
goog.require('Blockly.geras.RenderInfo');
goog.require('Blockly.geras.Drawer');
goog.require('Blockly.thrasos.RenderInfo');
goog.require('Blockly.zelos.RenderInfo');
@@ -68,13 +69,13 @@ Blockly.blockRendering.render = function(block) {
}
if (Blockly.blockRendering.rendererName == 'geras') {
var info = new Blockly.geras.RenderInfo(block);
new Blockly.geras.Drawer(block, info).draw_();
new Blockly.geras.Drawer(block, info).draw();
} else if (Blockly.blockRendering.rendererName == 'thrasos') {
var info = new Blockly.thrasos.RenderInfo(block);
new Blockly.blockRendering.Drawer(block, info).draw_();
new Blockly.blockRendering.Drawer(block, info).draw();
} else if (Blockly.blockRendering.rendererName == 'zelos') {
var info = new Blockly.zelos.RenderInfo(block);
new Blockly.blockRendering.Drawer(block, info).draw_();
new Blockly.blockRendering.Drawer(block, info).draw();
}
};

View File

@@ -44,8 +44,8 @@ goog.require('Blockly.geras.RenderInfo');
Blockly.geras.Drawer = function(block, info) {
Blockly.geras.Drawer.superClass_.constructor.call(this, block, info);
// Unlike Thrasos, Geras has highlights and drop shadows.
this.highlighter_ =
new Blockly.geras.Highlighter(this.info_, this.pathObject_);
this.highlighter_ = new Blockly.geras.Highlighter(
/** @type {!Blockly.geras.RenderInfo} */ (this.info_), this.pathObject_);
};
goog.inherits(Blockly.geras.Drawer, Blockly.blockRendering.Drawer);
@@ -89,7 +89,6 @@ Blockly.geras.Drawer.prototype.drawValueInput_ = function(row) {
Blockly.geras.Drawer.superClass_.drawValueInput_.call(this, row);
};
/**
* @override
*/
@@ -107,7 +106,6 @@ Blockly.geras.Drawer.prototype.drawRightSideRow_ = function(row) {
Blockly.geras.Drawer.superClass_.drawRightSideRow_.call(this, row);
};
/**
* @override
*/
@@ -128,7 +126,6 @@ Blockly.geras.Drawer.prototype.drawLeft_ = function() {
Blockly.geras.Drawer.superClass_.drawLeft_.call(this);
};
/**
* @override
*/