Bug 255477

Summary: REGRESSION (Safari 16.4): Max-height: 100% max-width:100% on image does not resize and layout image correctly
Product: WebKit Reporter: Alex G <garipian>
Component: CSSAssignee: Nobody <webkit-unassigned>
Status: RESOLVED DUPLICATE    
Severity: Blocker CC: sabouhallawa, sgill26, simon.fraser, webkit-bug-importer, zalan
Priority: P1 Keywords: InRadar
Version: Safari 16   
Hardware: Mac (Apple Silicon)   
OS: macOS 13   
URL: https://codesandbox.io/s/hungry-snow-9lvxuy?file=/index.html&resolutionWidth=464&resolutionHeight=240
Attachments:
Description Flags
all4browsers
none
STP 168 none

Alex G
Reported 2023-04-14 20:29:25 PDT
Created attachment 465922 [details] all4browsers Max-height: 100% max-width:100% on an image are not working correctly. Test page: https://codesandbox.io/s/hungry-snow-9lvxuy?file=/index.html&resolutionWidth=464&resolutionHeight=240 On the test page make sure height of browser is smaller than height of image. The url above has that set already. In safari 16.4 the image is not sized or laid out correctly. I expect the image height to fit the height of the browser and or container when max-width=100% and max-height=100% are set. This works fine in Safari 16.3 and below as well as latest Chrome and Firefox.
Attachments
all4browsers (693.28 KB, image/jpeg)
2023-04-14 20:29 PDT, Alex G 		no flags
Details
STP 168 (265.23 KB, image/png)
2023-04-19 14:32 PDT, alan 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2023-04-14 22:08:30 PDT
<rdar://problem/108078984>
alan
Comment 2 2023-04-15 07:09:12 PDT
This has regressed at 259663@main but trunk has already recovered at 262342@main.
alan
Comment 3 2023-04-15 07:12:03 PDT
Thank you for filing this bug report! I am going to mark it as duplicate as the attached test case seems to be working on trunk now.
alan
Comment 4 2023-04-15 07:12:23 PDT
*** This bug has been marked as a duplicate of bug 254603 ***
alan
Comment 5 2023-04-19 14:32:23 PDT
Created attachment 465990 [details] STP 168 This looks to be fixed in STP 168.
Alex G
Comment 6 2023-04-20 08:48:02 PDT
Thank you! I just tested with Safari TP and it resolved the regression on our website as well as the codesandbox link. Any eta on when it will be available for everyone to use?
Note You need to log in before you can comment on or make changes to this bug.