Closed Bug 1790616 Opened 2 years ago Closed 2 years ago

Use modern flexbox in the browser toolbox.

Categories

(Firefox :: Theme, task)

task

Tracking

()

RESOLVED FIXED
108 Branch
Tracking Status
firefox108 --- fixed

People

(Reporter: emilio, Assigned: emilio)

References

(Blocks 1 open bug)

Details

Attachments

(2 files, 1 obsolete file)

No description provided.

The min-width / contain are as usual to allow elements to shrink under
their intrinsic size. You can only see its effects with relatively thin
windows (or with a very large number of tabs in the tabstrip case).

Making it work in the browser area is blocked on making splitters work
on non-XUL boxes (so that DevTools can be resized), or replacing those
splitters with something else. I'll look at that next.

See Also: → 1710092
Attachment #9294445 - Attachment is obsolete: true
Depends on: 1793689
Depends on: 1793700
Depends on: 1793727
Depends on: 1793819
Depends on: 1794630

These fix rendering or test issues with flex emulation, but don't change
rendering without, so can land separately.

Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7d7d8d037b0d
Minor CSS tweaks that are needed for modern flexbox. r=dao
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/fb32775d2def
Use modern flexbox for the toolbox. r=dao,Gijs
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 107 Branch
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Status: REOPENED → RESOLVED
Closed: 2 years ago2 years ago
Resolution: --- → FIXED
Regressions: 1795260
Regressions: 1795286
Regressions: 1795294
Regressions: 1795315
Regressions: 1795316
Regressions: 1795335
Regressions: 1795339

Backed out changeset fb32775d2def as requested by dev. Backout link: https://hg.mozilla.org/integration/autoland/rev/4a24cecdad7190433bc76f853a9519c1e1e63447

Flags: needinfo?(emilio)
Status: RESOLVED → REOPENED
Flags: needinfo?(emilio)
Resolution: FIXED → ---
Regressions: 1795363
Backout by nfay@mozilla.com:
https://hg.mozilla.org/mozilla-central/rev/984b9b9d0404
Backed out changeset fb32775d2def as requested by dev a=graft
See Also: 1710092
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/aa325161aae5
Use modern flexbox for the toolbox. r=dao,Gijs
Status: REOPENED → RESOLVED
Closed: 2 years ago2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 108 Branch

Just updated to today's build after the landing of this path, the issue of huge drop lists is back, so the problem is not fixed.
crash report: Firefox 108.0a1 Crash Report [@ webrender::render_api::window_size_sanity_check ]

Build ID 20221018094831
change set; https://hg.mozilla.org/mozilla-central/rev/826073b561e026d8fbc6a8354081db59727d9f33

bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1705002

(In reply to Jim Jeffery not reading bug-mail 1/2/11 from comment #13)

Just updated to today's build after the landing of this path, the issue of huge drop lists is back, so the problem is not fixed.
crash report: Firefox 108.0a1 Crash Report [@ webrender::render_api::window_size_sanity_check ]

Build ID 20221018094831
change set; https://hg.mozilla.org/mozilla-central/rev/826073b561e026d8fbc6a8354081db59727d9f33

A little more context in bug 1705002 - Jim, can you link to your crash report on crash-stats please?

Flags: needinfo?(jmjjeffery)

(In reply to Jim Jeffery not reading bug-mail 1/2/11 from comment #13)

Just updated to today's build after the landing of this path, the issue of huge drop lists is back, so the problem is not fixed.
crash report: Firefox 108.0a1 Crash Report [@ webrender::render_api::window_size_sanity_check ]

Which menu and on which OS?

(In reply to Emilio Cobos Álvarez (:emilio) from comment #15)

(In reply to Jim Jeffery not reading bug-mail 1/2/11 from comment #13)

Just updated to today's build after the landing of this path, the issue of huge drop lists is back, so the problem is not fixed.
crash report: Firefox 108.0a1 Crash Report [@ webrender::render_api::window_size_sanity_check ]

Which menu and on which OS?

The list-box is on a site that requires a login and being a member to the ARRL (Amateur Radio Relay League)
There is an option to 'find call' that you can check if they use the logbook on the ARRL site. That list-box usually lists the last 6 lookups.
The list is now over a full page tall and after 2-3 seconds crashes the entire browser.

Flags: needinfo?(jmjjeffery)

Oh, so a <select> menulist? That's useful, can try to repro. Also, what about the OS? Is this on Linux, macOS or Windows? Thanks again.

(In reply to :Gijs (he/him) from comment #14)

(In reply to Jim Jeffery not reading bug-mail 1/2/11 from comment #13)

Just updated to today's build after the landing of this path, the issue of huge drop lists is back, so the problem is not fixed.
crash report: Firefox 108.0a1 Crash Report [@ webrender::render_api::window_size_sanity_check ]

Build ID 20221018094831
change set; https://hg.mozilla.org/mozilla-central/rev/826073b561e026d8fbc6a8354081db59727d9f33

A little more context in bug 1705002 - Jim, can you link to your crash report on crash-stats please?

Not sure how to link crash report, here are links to my crashes:
https://crash-stats.mozilla.org/report/index/558b1630-d095-4e70-a15a-5da060221018
https://crash-stats.mozilla.org/report/index/dccbe237-549d-4f85-98f9-f932a0221018
https://crash-stats.mozilla.org/report/index/af3a92db-dbe1-47ab-88f4-c80f30221018

(In reply to Emilio Cobos Álvarez (:emilio) from comment #17)

Oh, so a <select> menulist? That's useful, can try to repro. Also, what about the OS? Is this on Linux, macOS or Windows? Thanks again.

Oh, sorry - Windows 11 Home 22h2, fall release update for latest windows.

Can't reproduce that with something like this, which creates a very tall <select> list. The list is properly scrollable on macOS and Linux, at least, will check Windows in a bit:

<!doctype html>
<select id="s">
  <option>ABC</option>
</select>
<script>
  s.options.length = 9999;
</script>

Can't reproduce on windows, tried ~60k different options, and also I tried to make them really wide as well, with no avail. If you could privately mail me a way of accessing that page I'd be happy to dig into it, it may be due to some styling they use?

(In reply to Emilio Cobos Álvarez (:emilio) from comment #21)

Can't reproduce on windows, tried ~60k different options, and also I tried to make them really wide as well, with no avail. If you could privately mail me a way of accessing that page I'd be happy to dig into it, it may be due to some styling they use?

Email sent.

P.S. I don't know how to clear the list box on that site. The info there is not important so it could be deleted/removed or whatever, but I don't know how.

Regressions: 1795890
Regressions: 1795892
Regressions: 1795933
Regressions: 1795934
Regressions: 1795938
Blocks: 1795944
Regressions: 1795986
Regressions: 1796040
Regressions: 1796045
Regressions: 1796137
Regressions: 1796138
Regressions: 1796179
Regressions: 1796698
Regressions: 1796735
Regressions: 1796944
Regressions: 1797052
Regressions: 1797249
Regressions: 1797272
Regressions: 1797432
Regressions: 1796065
Regressions: 1799460
Regressions: 1799491
Regressions: 1801502
Regressions: 1802722
Regressions: 1806518
Regressions: 1812445
Regressions: 1812636
Blocks: 1604427
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: