RESOLVED FIXED Bug 135335
`resize` treats `width` and `height` as minimum dimensions
https://bugs.webkit.org/show_bug.cgi?id=135335
Summary `resize` treats `width` and `height` as minimum dimensions
Lea Verou
Reported 2014-07-27 21:58:39 PDT
(Surprised this isn’t already reported but did a lot of searching and couldn’t find anything so filing just in case. Sorry if dupe!) Yes, I know that this is technically allowed in css3-ui ("The user agent may restrict the resizing range to something suitable, such as between the original formatted size of the element, and large enough to encompass all the element's contents."), but this behavior is seriously crippling the usefulness of the resize property. A quick search reveals quite a few authors scratching their heads about this issue, for example (top 5): http://stackoverflow.com/questions/12985133/resize-a-div-to-smaller-than-its-declared-size http://stackoverflow.com/questions/18178301/how-can-i-use-css-resize-to-resize-an-element-to-a-height-width-less-than-init http://stackoverflow.com/questions/15108716/how-to-resize-div-to-less-than-its-initial-width-height http://stackoverflow.com/questions/7197136/using-css-to-resize-a-div-to-a-dimension-smaller-than-its-current-width-and-he http://stackoverflow.com/questions/15766784/css-resize-min-width Firefox only restricts it to the min-width/min-height, or if those are not set, to the dimensions of the resizer (roughly 15px by 15px). Chrome does the same as WebKit, but it’s a confirmed bug they are about to fix: https://code.google.com/p/chromium/issues/detail?can=2&start=0&num=100&q=&colspec=ID%20Pri%20M%20Iteration%20ReleaseBlock%20Cr%20Status%20Owner%20Summary%20OS%20Modified&groupby=&sort=&id=94583
Attachments
Patch (23.96 KB, patch)
2022-04-07 04:35 PDT, Tim Nguyen (:ntim)
no flags
Patch (24.87 KB, patch)
2022-04-07 04:39 PDT, Tim Nguyen (:ntim)
no flags
Patch (31.48 KB, patch)
2022-04-07 12:35 PDT, Tim Nguyen (:ntim)
no flags
Patch (31.73 KB, patch)
2022-04-07 12:38 PDT, Tim Nguyen (:ntim)
simon.fraser: review+
simon.fraser: commit-queue-
[fast-cq] Patch (31.71 KB, patch)
2022-04-07 13:18 PDT, Tim Nguyen (:ntim)
no flags
Sam
Comment 1 2019-09-08 14:39:12 PDT
This is still unfixed in Safari
Radar WebKit Bug Importer
Comment 2 2019-09-09 11:16:28 PDT
Simon Fraser (smfr)
Comment 3 2021-08-26 17:41:54 PDT
Simon Fraser (smfr)
Comment 4 2021-08-26 20:30:07 PDT
Michal
Comment 5 2022-03-04 08:01:33 PST
Hello, is this issue planned to fix it? I would really appreciate it. Thank you for your answer. :)
Tim Nguyen (:ntim)
Comment 6 2022-04-06 21:41:26 PDT
I'll take this.
Tim Nguyen (:ntim)
Comment 7 2022-04-07 04:35:23 PDT
Tim Nguyen (:ntim)
Comment 8 2022-04-07 04:39:21 PDT
Tim Nguyen (:ntim)
Comment 9 2022-04-07 06:28:08 PDT
Comment on attachment 456909 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=456909&action=review > LayoutTests/fast/css/resize-orthogonal-containing-block.html:45 > +}, "Test for resizing the TEXTAREA below its initial size and with orthogonal containing block."); It should say "the box"
Simon Fraser (smfr)
Comment 10 2022-04-07 10:02:31 PDT
Comment on attachment 456909 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=456909&action=review > Source/WebCore/rendering/RenderLayer.cpp:2760 > + // Match Firefox and Chrome, you cannot resize below this value. > + int strictMinimumSize = 15; Does this work if there are custom scrollbars with a larger width/height? Seems like this needs to be some minimum size plus non-overlay scrollbar width.
Tim Nguyen (:ntim)
Comment 11 2022-04-07 12:35:26 PDT
Tim Nguyen (:ntim)
Comment 12 2022-04-07 12:38:05 PDT
Simon Fraser (smfr)
Comment 13 2022-04-07 13:08:59 PDT
Comment on attachment 456954 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=456954&action=review > Source/WebCore/rendering/RenderLayer.cpp:2750 > + auto resizerRect = ensureLayerScrollableArea()->overflowControlsRects().resizer; If this element isn't scrollable we shouldn't trigger creation of a RenderLayerScrollableArea here. Also overflowControlsRects() exists on RenderLayer so we should just call that.
Tim Nguyen (:ntim)
Comment 14 2022-04-07 13:18:28 PDT
Created attachment 456962 [details] [fast-cq] Patch
Tim Nguyen (:ntim)
Comment 15 2022-04-07 13:26:00 PDT
Comment on attachment 456962 [details] [fast-cq] Patch I'll land this myself
Tim Nguyen (:ntim)
Comment 16 2022-04-07 13:26:14 PDT
Simon Fraser (smfr)
Comment 17 2022-08-18 09:46:42 PDT
*** Bug 72948 has been marked as a duplicate of this bug. ***
Note You need to log in before you can comment on or make changes to this bug.