mirror of
https://github.com/google/blockly.git
synced 2026-01-12 03:17:09 +01:00
Remove firefox workaround that is no longer needed (#3349)
This commit is contained in:
@@ -333,30 +333,6 @@ Blockly.VerticalFlyout.prototype.getClientRect = function() {
|
||||
var width = flyoutRect.width;
|
||||
return new Blockly.utils.Rect(-BIG_NUM, BIG_NUM, -BIG_NUM, left + width);
|
||||
} else { // Right
|
||||
// Firefox sometimes reports the wrong value for the client rect.
|
||||
// See https://github.com/google/blockly/issues/1425 and
|
||||
// https://bugzilla.mozilla.org/show_bug.cgi?id=1066435
|
||||
if (Blockly.utils.userAgent.GECKO &&
|
||||
this.targetWorkspace_ && this.targetWorkspace_.isMutator) {
|
||||
// The position of the left side of the mutator workspace in pixels
|
||||
// relative to the window origin.
|
||||
var targetWsLeftPixels =
|
||||
this.targetWorkspace_.svgGroup_.getBoundingClientRect().x;
|
||||
// The client rect is in pixels relative to the window origin. When the
|
||||
// browser gets the wrong value it reports that the flyout left is the
|
||||
// same as the mutator workspace left.
|
||||
// We know that in a mutator workspace with the flyout on the right, the
|
||||
// visible area of the workspace should be more than ten pixels wide. If
|
||||
// the browser reports that the flyout is within ten pixels of the left
|
||||
// side of the workspace, ignore it and manually calculate the value.
|
||||
if (Math.abs(targetWsLeftPixels - left) < 10) {
|
||||
// If we're in a mutator, its scale is always 1, purely because of some
|
||||
// oddities in our rendering optimizations. The actual scale is the
|
||||
// same as the scale on the parent workspace.
|
||||
var scale = this.targetWorkspace_.options.parentWorkspace.scale;
|
||||
left += this.leftEdge_ * scale;
|
||||
}
|
||||
}
|
||||
return new Blockly.utils.Rect(-BIG_NUM, BIG_NUM, left, BIG_NUM);
|
||||
}
|
||||
};
|
||||
|
||||
Reference in New Issue
Block a user