Skip to main content

Search and Top Navigation

#15226 new bug ()

Opened August 10, 2017 05:04AM UTC

Last modified August 10, 2017 07:35AM UTC

style.height being forced on canvas after dragging element on recent Chrome versions

Reported by: hexalys Owned by:
Priority: minor Milestone: none
Component: ui.draggable Version: 1.12.1
Keywords: Cc:
Blocked by: Blocking:
Description

I have a product builder using Draggable (https://pridesash.com/builder/#step/1) which used to work fine with UI 1.10 or under. I now suddenly find myself with a bug on newer chrome versions since using UI 1.11.4 (I think).

When dragging a canvas element, for some reason, jQueryUI now sets a css style 'height' on Chrome 59+ (at least) in addition to 'left' and 'top'. That 'height' isn't set on Chrome 49, Safari, Firefox or Edge. Only recent Chrome versions. I assume this 'height' might be related to a different unrelated chrome bug.

I was relying only on the 'width' and 'height' attributes of the canvas for the canvas proportions. This new unnecessary introduced 'height' messes up proportions of canvas boxes when re-editing a piece of canvas text.

This now forces me to set css style 'height' and 'width' every time to prevent the ui 'height' from affecting canvas proportions and either shrinking or expanding the box width. But that shouldn't be required.

Any clue has to what this new 'height' comes from?

And could it be removed so canvas proportion aren't affected by it?

For the REPRO: You can test at: https://pridesash.com/builder/ with the following steps:

Select a sash, open the chrome inspector, and on Step 2, drag the "Class of 2017" box down. You will see that the original canvas element height I now set in subpixel (67.35ish px for patching the bug with both width and height) will turn into 67px as set by jQuery UI. Which only happens on newest Chrome.

Attachments (0)
Change History (5)

Changed August 10, 2017 05:21AM UTC by hexalys comment:1

description: I have a product builder using Draggable (https://pridesash.com/builder/#step/1) which used to work fine with UI 1.10 or under. I now suddenly find myself with a bug on newer chrome versions since using UI 1.11.4 (I think). \ \ When dragging a canvas element, for some reason, jQueryUI now sets a css style 'height' on Chrome 59+ (at least) in addition to 'left' and 'top'. That 'height' isn't set on Chrome 49, Safari, Firefox or Edge. Only recent Chrome versions. I assume this 'height' might be related to a different unrelated chrome bug. \ \ I was relying only on the 'width' and 'height' attributes of the canvas for the canvas proportions. This new unnecessary introduced 'height' messes up proportions of canvas boxes when re-editing a piece of canvas text. \ \ This now forces me to set css style 'height' and 'width' every time to prevent the 'height' from affecting canvas proportions and either shrinking or expanding the box width. But that shouldn't be required. \ \ Any clue has to what this new 'height' comes from? \ \ And could it be removed so canvas proportion aren't affected by it?I have a product builder using Draggable (https://pridesash.com/builder/#step/1) which used to work fine with UI 1.10 or under. I now suddenly find myself with a bug on newer chrome versions since using UI 1.11.4 (I think). \ \ When dragging a canvas element, for some reason, jQueryUI now sets a css style 'height' on Chrome 59+ (at least) in addition to 'left' and 'top'. That 'height' isn't set on Chrome 49, Safari, Firefox or Edge. Only recent Chrome versions. I assume this 'height' might be related to a different unrelated chrome bug. \ \ I was relying only on the 'width' and 'height' attributes of the canvas for the canvas proportions. This new unnecessary introduced 'height' messes up proportions of canvas boxes when re-editing a piece of canvas text. \ \ This now forces me to set css style 'height' and 'width' every time to prevent the 'height' from affecting canvas proportions and either shrinking or expanding the box width. But that shouldn't be required. \ \ Any clue has to what this new 'height' comes from? \ And could it be removed so canvas proportion aren't affected by it? \ \ For the REPRO: You can test at: https://pridesash.com/builder/ with the following steps: \ \ Select a sash, open the chrome inspector, and on Step 2, drag the "Class of 2017" box down. You will see that the original height I set in subpixel (67.35ish) will turn into 67px as set by jQuery UI. Which only happens on newest Chrome.

Changed August 10, 2017 05:23AM UTC by hexalys comment:2

description: I have a product builder using Draggable (https://pridesash.com/builder/#step/1) which used to work fine with UI 1.10 or under. I now suddenly find myself with a bug on newer chrome versions since using UI 1.11.4 (I think). \ \ When dragging a canvas element, for some reason, jQueryUI now sets a css style 'height' on Chrome 59+ (at least) in addition to 'left' and 'top'. That 'height' isn't set on Chrome 49, Safari, Firefox or Edge. Only recent Chrome versions. I assume this 'height' might be related to a different unrelated chrome bug. \ \ I was relying only on the 'width' and 'height' attributes of the canvas for the canvas proportions. This new unnecessary introduced 'height' messes up proportions of canvas boxes when re-editing a piece of canvas text. \ \ This now forces me to set css style 'height' and 'width' every time to prevent the 'height' from affecting canvas proportions and either shrinking or expanding the box width. But that shouldn't be required. \ \ Any clue has to what this new 'height' comes from? \ And could it be removed so canvas proportion aren't affected by it? \ \ For the REPRO: You can test at: https://pridesash.com/builder/ with the following steps: \ \ Select a sash, open the chrome inspector, and on Step 2, drag the "Class of 2017" box down. You will see that the original height I set in subpixel (67.35ish) will turn into 67px as set by jQuery UI. Which only happens on newest Chrome.I have a product builder using Draggable (https://pridesash.com/builder/#step/1) which used to work fine with UI 1.10 or under. I now suddenly find myself with a bug on newer chrome versions since using UI 1.11.4 (I think). \ \ When dragging a canvas element, for some reason, jQueryUI now sets a css style 'height' on Chrome 59+ (at least) in addition to 'left' and 'top'. That 'height' isn't set on Chrome 49, Safari, Firefox or Edge. Only recent Chrome versions. I assume this 'height' might be related to a different unrelated chrome bug. \ \ I was relying only on the 'width' and 'height' attributes of the canvas for the canvas proportions. This new unnecessary introduced 'height' messes up proportions of canvas boxes when re-editing a piece of canvas text. \ \ This now forces me to set css style 'height' and 'width' every time to prevent the 'height' from affecting canvas proportions and either shrinking or expanding the box width. But that shouldn't be required. \ \ Any clue has to what this new 'height' comes from? \ And could it be removed so canvas proportion aren't affected by it? \ \ For the REPRO: You can test at: https://pridesash.com/builder/ with the following steps: \ \ Select a sash, open the chrome inspector, and on Step 2, drag the "Class of 2017" box down. You will see that the original height I set in subpixel (67.35ish patchign the bug with both width and height set) will turn into 67px as set by jQuery UI. Which only happens on newest Chrome.

Changed August 10, 2017 05:24AM UTC by hexalys comment:3

description: I have a product builder using Draggable (https://pridesash.com/builder/#step/1) which used to work fine with UI 1.10 or under. I now suddenly find myself with a bug on newer chrome versions since using UI 1.11.4 (I think). \ \ When dragging a canvas element, for some reason, jQueryUI now sets a css style 'height' on Chrome 59+ (at least) in addition to 'left' and 'top'. That 'height' isn't set on Chrome 49, Safari, Firefox or Edge. Only recent Chrome versions. I assume this 'height' might be related to a different unrelated chrome bug. \ \ I was relying only on the 'width' and 'height' attributes of the canvas for the canvas proportions. This new unnecessary introduced 'height' messes up proportions of canvas boxes when re-editing a piece of canvas text. \ \ This now forces me to set css style 'height' and 'width' every time to prevent the 'height' from affecting canvas proportions and either shrinking or expanding the box width. But that shouldn't be required. \ \ Any clue has to what this new 'height' comes from? \ And could it be removed so canvas proportion aren't affected by it? \ \ For the REPRO: You can test at: https://pridesash.com/builder/ with the following steps: \ \ Select a sash, open the chrome inspector, and on Step 2, drag the "Class of 2017" box down. You will see that the original height I set in subpixel (67.35ish patchign the bug with both width and height set) will turn into 67px as set by jQuery UI. Which only happens on newest Chrome.I have a product builder using Draggable (https://pridesash.com/builder/#step/1) which used to work fine with UI 1.10 or under. I now suddenly find myself with a bug on newer chrome versions since using UI 1.11.4 (I think). \ \ When dragging a canvas element, for some reason, jQueryUI now sets a css style 'height' on Chrome 59+ (at least) in addition to 'left' and 'top'. That 'height' isn't set on Chrome 49, Safari, Firefox or Edge. Only recent Chrome versions. I assume this 'height' might be related to a different unrelated chrome bug. \ \ I was relying only on the 'width' and 'height' attributes of the canvas for the canvas proportions. This new unnecessary introduced 'height' messes up proportions of canvas boxes when re-editing a piece of canvas text. \ \ This now forces me to set css style 'height' and 'width' every time to prevent the 'height' from affecting canvas proportions and either shrinking or expanding the box width. But that shouldn't be required. \ \ Any clue has to what this new 'height' comes from? \ And could it be removed so canvas proportion aren't affected by it? \ \ For the REPRO: You can test at: https://pridesash.com/builder/ with the following steps: \ \ Select a sash, open the chrome inspector, and on Step 2, drag the "Class of 2017" box down. You will see that the original height I now set in subpixel (67.35ish px for patching the bug with both width and height) will turn into 67px as set by jQuery UI. Which only happens on newest Chrome.

Changed August 10, 2017 05:26AM UTC by hexalys comment:4

description: I have a product builder using Draggable (https://pridesash.com/builder/#step/1) which used to work fine with UI 1.10 or under. I now suddenly find myself with a bug on newer chrome versions since using UI 1.11.4 (I think). \ \ When dragging a canvas element, for some reason, jQueryUI now sets a css style 'height' on Chrome 59+ (at least) in addition to 'left' and 'top'. That 'height' isn't set on Chrome 49, Safari, Firefox or Edge. Only recent Chrome versions. I assume this 'height' might be related to a different unrelated chrome bug. \ \ I was relying only on the 'width' and 'height' attributes of the canvas for the canvas proportions. This new unnecessary introduced 'height' messes up proportions of canvas boxes when re-editing a piece of canvas text. \ \ This now forces me to set css style 'height' and 'width' every time to prevent the 'height' from affecting canvas proportions and either shrinking or expanding the box width. But that shouldn't be required. \ \ Any clue has to what this new 'height' comes from? \ And could it be removed so canvas proportion aren't affected by it? \ \ For the REPRO: You can test at: https://pridesash.com/builder/ with the following steps: \ \ Select a sash, open the chrome inspector, and on Step 2, drag the "Class of 2017" box down. You will see that the original height I now set in subpixel (67.35ish px for patching the bug with both width and height) will turn into 67px as set by jQuery UI. Which only happens on newest Chrome.I have a product builder using Draggable (https://pridesash.com/builder/#step/1) which used to work fine with UI 1.10 or under. I now suddenly find myself with a bug on newer chrome versions since using UI 1.11.4 (I think). \ \ When dragging a canvas element, for some reason, jQueryUI now sets a css style 'height' on Chrome 59+ (at least) in addition to 'left' and 'top'. That 'height' isn't set on Chrome 49, Safari, Firefox or Edge. Only recent Chrome versions. I assume this 'height' might be related to a different unrelated chrome bug. \ \ I was relying only on the 'width' and 'height' attributes of the canvas for the canvas proportions. This new unnecessary introduced 'height' messes up proportions of canvas boxes when re-editing a piece of canvas text. \ \ This now forces me to set css style 'height' and 'width' every time to prevent the ui 'height' from affecting canvas proportions and either shrinking or expanding the box width. But that shouldn't be required. \ \ Any clue has to what this new 'height' comes from? \ And could it be removed so canvas proportion aren't affected by it? \ \ For the REPRO: You can test at: https://pridesash.com/builder/ with the following steps: \ \ Select a sash, open the chrome inspector, and on Step 2, drag the "Class of 2017" box down. You will see that the original height I now set in subpixel (67.35ish px for patching the bug with both width and height) will turn into 67px as set by jQuery UI. Which only happens on newest Chrome.

Changed August 10, 2017 07:35AM UTC by hexalys comment:5

description: I have a product builder using Draggable (https://pridesash.com/builder/#step/1) which used to work fine with UI 1.10 or under. I now suddenly find myself with a bug on newer chrome versions since using UI 1.11.4 (I think). \ \ When dragging a canvas element, for some reason, jQueryUI now sets a css style 'height' on Chrome 59+ (at least) in addition to 'left' and 'top'. That 'height' isn't set on Chrome 49, Safari, Firefox or Edge. Only recent Chrome versions. I assume this 'height' might be related to a different unrelated chrome bug. \ \ I was relying only on the 'width' and 'height' attributes of the canvas for the canvas proportions. This new unnecessary introduced 'height' messes up proportions of canvas boxes when re-editing a piece of canvas text. \ \ This now forces me to set css style 'height' and 'width' every time to prevent the ui 'height' from affecting canvas proportions and either shrinking or expanding the box width. But that shouldn't be required. \ \ Any clue has to what this new 'height' comes from? \ And could it be removed so canvas proportion aren't affected by it? \ \ For the REPRO: You can test at: https://pridesash.com/builder/ with the following steps: \ \ Select a sash, open the chrome inspector, and on Step 2, drag the "Class of 2017" box down. You will see that the original height I now set in subpixel (67.35ish px for patching the bug with both width and height) will turn into 67px as set by jQuery UI. Which only happens on newest Chrome.I have a product builder using Draggable (https://pridesash.com/builder/#step/1) which used to work fine with UI 1.10 or under. I now suddenly find myself with a bug on newer chrome versions since using UI 1.11.4 (I think). \ \ When dragging a canvas element, for some reason, jQueryUI now sets a css style 'height' on Chrome 59+ (at least) in addition to 'left' and 'top'. That 'height' isn't set on Chrome 49, Safari, Firefox or Edge. Only recent Chrome versions. I assume this 'height' might be related to a different unrelated chrome bug. \ \ I was relying only on the 'width' and 'height' attributes of the canvas for the canvas proportions. This new unnecessary introduced 'height' messes up proportions of canvas boxes when re-editing a piece of canvas text. \ \ This now forces me to set css style 'height' and 'width' every time to prevent the ui 'height' from affecting canvas proportions and either shrinking or expanding the box width. But that shouldn't be required. \ \ Any clue has to what this new 'height' comes from? \ And could it be removed so canvas proportion aren't affected by it? \ \ For the REPRO: You can test at: https://pridesash.com/builder/ with the following steps: \ \ Select a sash, open the chrome inspector, and on Step 2, drag the "Class of 2017" box down. You will see that the original canvas element height I now set in subpixel (67.35ish px for patching the bug with both width and height) will turn into 67px as set by jQuery UI. Which only happens on newest Chrome.