Closed Bug 1501207 Opened 3 years ago Closed 3 years ago
The flex item sizing property section sometimes says that an item was "Set to grow" and "Not set to grow" at the same time
46 bytes, text/x-phabricator-request
|Details | Review|
STR: - open the following test case URL: data:text/html,<div style="display:flex;width:500px;border:1px solid;"><span style="flex-grow:1;flex-basis:100px;max-width:50px;">item 1</span><span style="flex-grow:1">item 2</span></div> - open the inspector - select the first <span> flex item within the flex container --> The flex item sizing property section says: > There was extra room available on the flex line. > Item was set to grow. > Item was not set to grow. > Item could not grow, siblings have likely used the extra space. We should fix it so that it doesn't say that it was set *and* not set to grow at the same time.
As noted in bug 1501066, we have the same issue with "set to shrink" + "not set to shrink" being simultaneously reported for the same item. (Addressing the "shrink" messaging might be out of scope for this bug, if you want to focus just on "set to grow" here for simplicity's sake. Either way, I'm mentioning it here in case it influences any decisions around design/testcases/anything-else in preparation for fixing that other side of the issue.)
Thanks Daniel, I have seen that other bug you filed. I intend to look at that one next. For now, I'm addressing these issues one by one, separately, but I have a feeling that at some stage, the whole "flexibility" logic of the flexbox inspector will need some refactoring. Not yet though.
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/c4123efb0536 Don't say that an item was both set and not set to grow; r=jdescottes
You need to log in before you can comment on or make changes to this bug.