Closed Bug 1792431 Opened 2 years ago Closed 2 years ago

[Windows] Always open similar files check mark is not displayed in about:downloads

Categories

(Firefox :: Downloads Panel, defect, P3)

Desktop
All
defect

Tracking

()

VERIFIED FIXED
112 Branch
Tracking Status
firefox-esr102 --- wontfix
firefox105 --- wontfix
firefox106 --- wontfix
firefox107 --- wontfix
firefox112 --- verified

People

(Reporter: atrif, Assigned: emilio)

References

(Blocks 1 open bug)

Details

Attachments

(3 files)

Attached image checkmark_about_downloads.gif (deleted) —

Found in

  • 106.0b4 (20220925185751)

Affected versions

  • 106.0b4 (20220925185751)
  • 107.0a1 (20220925213821)
  • 105.0(20220915150737)
  • 102.3.0esr (20220912135840)

Tested platforms

  • Affected platforms: Windows 7x64, Ubuntu 20.04

  • Unaffected platforms: Windows 10x64, macOS 11

Steps to reproduce

  1. Download a random file like http://ipv4.download.thinkbroadband.com/5MB.zip.
  2. Open about:downloads page and right-click on the downloaded item.
  3. Click on Always open similar files.
  4. Right-click on the previously downloaded item again.

Expected result

  • Always open similar files option has a checkmark in front.

Actual result

  • No checkmark is displayed.

Regression range

  • I will search for one ASAP if there is one.

Additional notes

  • Attached a screen recording.
  • Windows 10x64 and macOS 11 are not affected.
  • This happens in Private Windows as well.
Has STR: --- → yes
QA Whiteboard: [qa-regression-triage]

This is not a regression. Tried searching for a good and bad build using mozregression but the only good builds are the ones in which this functionality is not available.

No longer blocks: 1740701
QA Whiteboard: [qa-regression-triage]
Severity: S4 → S3
Priority: -- → P3

I can repro on more modern Windows if I enable native menus using the about:config pref (browser.display.windows.non_native_menus to 0).

I... don't understand why it doesn't work? It seems to use type=checkbox and checked=true the same way that e.g. the "menu bar" item in the toolbar context menu uses, and that works fine. And I don't see any difference when looking at styles for those two elements. Maybe it's because it's not in the toplevel document or something? Emilio, can you help?

Flags: needinfo?(emilio)
Assignee: nobody → emilio
Status: NEW → ASSIGNED

about:downloads is in a content dochsell, which means it uses the non-native theme... I have a patch that ought to work, but untested because I'm on Linux right now.

Flags: needinfo?(emilio)
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/900fc2b830da Use non-native menus in Windows for documents where we don't have a native theme. r=Gijs
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch
Attached image image.png (deleted) —

Confirming that the issue is fixed on Windows 7x64 and Windows 10x64 (browser.display.windows.non_native_menus:0) with Firefox 112.0a1 (20230216213942) and macOS 12 is still not affected.

However, I can still reproduce the issue on Ubuntu 20.04 with the latest Nightly, should I file a new ticket for this?

Also, I can see that there are some differences in the alignment of the context menu options when the check mark is active between Windows 7 and Windows 10/ macOS 12. Should we file an issue for this as well? Thank you in advance!

Flags: needinfo?(emilio)

Yeah, let's file a new bug for Linux, we don't have non-native checkboxes there unfortunately, so that requires more work.

I wouldn't worry about the alignment differences across platforms, I don't think. But 301 Gijs on that.

Flags: needinfo?(emilio) → needinfo?(gijskruitbosch+bugs)

Thank you, Emilio. I filled bug 1817422 for Ubuntu and I will fill another if needed for the alignment differences.

Status: RESOLVED → VERIFIED
Summary: Always open similar files check mark is not displayed in about:downloads → [Windows] Always open similar files check mark is not displayed in about:downloads

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

Yeah, let's file a new bug for Linux, we don't have non-native checkboxes there unfortunately, so that requires more work.

I wouldn't worry about the alignment differences across platforms, I don't think. But 301 Gijs on that.

Huh. That's interesting. The cause is this code: https://searchfox.org/mozilla-central/rev/9de332d5c8faac58dc1232b8a6383ce6cb1400f4/toolkit/content/widgets/menupopup.js#14-34 .

It should use some other check now that we support these menus on other Windows versions. I'll file a follow-up.

Flags: needinfo?(gijskruitbosch+bugs)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: