#18360 closed bug (fixed)

Web+: Large images aren't downsized-to-fit window

Reported by: humdinger Owned by: pulkomandy
Priority: normal Milestone: R1/beta5
Component: Kits/Web Kit Version: R1/beta4
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description

This is hrev56578+77, 64bit. (HaikuWebKit 1.9.4)

Web+ used to downsize large images to fit in the current window. The mouse pointer became a magnifying glass and clicking the image popped the image to 100%. Click again and it got resized to fit the window again.

Can we have that back, please?

Change History (8)

comment:1 by nephele, 21 months ago

Was the behaviour you wanted working in 1.9.3 or has it been gone longer?

comment:2 by humdinger, 21 months ago

It was working before I last updated Beta4. I update my Beta4 install about every two weeks.

I just remembered that there was a similar regression 1.5 years ago: #17387
Maybe a hint where to look.

comment:3 by pulkomandy, 21 months ago

Yes, it looks like the change was lost when the file was renamed:

https://github.com/haiku/haikuwebkit/blob/eac2f26b8bf63caeefd16a7af0aef74adec22f06/Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml#L5949

So the same fix can be applied again.

comment:4 by madmax, 21 months ago

The InputType*Enabled were also lost, but activating them I get an instantly disappearing calendar for dates and a color picker that crashes when a color is accepted. ShrinksStandaloneImagesToFit does work.

comment:5 by nephele, 21 months ago

calendars were also broken before in that way iirc

comment:6 by pulkomandy, 21 months ago

That code does not get enough testing, I had it working when I first wrote it but then not many websites use it and it probably got broken in one of the rebases. May be better to keep them disabled for now, then?

comment:7 by nephele, 21 months ago

Yes, but probably with a ticket as a reminder to reenable them later :)

comment:8 by pulkomandy, 21 months ago

Milestone: UnscheduledR1/beta5
Resolution: fixed
Status: newclosed

Fixed in https://github.com/haiku/haikuwebkit/pull/29

Will be included in HaikuWebKit 1.9.5, I guess.

Note: See TracTickets for help on using tickets.