New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 633645 link

Starred by 2 users
Status: WontFix
Owner: ----
Closed: Aug 20
Cc:
cbiesin...@chromium.org
schenney@chromium.org
█ mbarowsky+blinkimage@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 3
Type: Bug



Sign in to add a comment

broken image placeholder does not scale correctly inside a flexbox

Project Member Reported by cbiesin...@chromium.org, Aug 2 2016 
http://jsbin.com/bodaleqowi/edit?html,output

Even though the <img> element is sized the same as the flexbox (as it should be), the placeholder borders are not stretched with it.

That's due to this code:
https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/html/HTMLImageFallbackHelper.cpp?q=alttext-container&sq=package:chromium&dr=C&l=91

    // If the image has specified dimensions allow the alt-text container expand to fill them.
    if (newStyle->width().isSpecifiedOrIntrinsic() && newStyle->height().isSpecifiedOrIntrinsic()) {
        placeHolder->setInlineStyleProperty(CSSPropertyWidth, 100, CSSPrimitiveValue::UnitType::Percentage);
        placeHolder->setInlineStyleProperty(CSSPropertyHeight, 100, CSSPrimitiveValue::UnitType::Percentage);
    }

This does not take flexbox into account.

Comment 1 by e...@chromium.org, Aug 2 2016

Components: -Blink>Layout Blink>Image
Status: Available (was: Untriaged)
Project Member

Comment 2 by sheriffbot@chromium.org, Aug 3 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 3 by schenney@chromium.org, Aug 4 2017

Cc: schenney@chromium.org
Labels: -Hotlist-Recharge-Cold BugSource-Chromium PaintTeamTriaged-20170804
Status: Available (was: Untriaged)
Might be an easy fix.
Project Member

Comment 4 by sheriffbot@chromium.org, Aug 6

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 5 by schenney@chromium.org, Aug 6

Cc: cbiesin...@chromium.org
Labels: Needs-Feedback OS-Android OS-Chrome OS-Linux OS-Mac OS-Windows
NextAction: 2018-08-20
Status: Unconfirmed (was: Untriaged)
Our behavior matches Firefox. Am I missing something here?

Comment 6 by monor...@bugs.chromium.org, Aug 20 

The NextAction date has arrived: 2018-08-20

Comment 7 by schenney@chromium.org, Aug 20

NextAction: ----
Still need to know what the expected behavior is. Closing. Will reopen if feedback is received.

Comment 8 by schenney@chromium.org, Aug 20

Status: WontFix (was: Unconfirmed)
►

Sign in to add a comment