black7375 / Firefox-UI-Fix

🦊 I respect proton UI and aim to improve it.
Mozilla Public License 2.0
5.24k stars 183 forks source link

Certain areas uses old profiling icons (about:profiling & Process Manager) #918

Closed Aleksandra0056 closed 2 weeks ago

Aleksandra0056 commented 2 weeks ago

Describe the bug

I noticed this much earlier, but now, a few months after Lepton icons re-design, I finally decided to report this issue here.

The about:profiling page uses a non-Lepton icon for the header image (because the mentioned icon belongs to older Photon icons).

I reported a similar issue on Bugzilla (1738529), but that doesn't apply to this since clean Firefox doesn't use Lepton icons.

More affected areas are explained in the comments below.


Appropriate Lepton icon for replacement (performance.svg):

performance

NOTE: That icon appears vertically mirrored in some areas for some reason.

Expected behavior

The about:profiling page should use the same icon (which is on Developer Ed. & Nightly toolbar) as an icon of a header image.

Screenshots

Screenshot of the image in about:profiling header:

The aboutProfiling page uses non-Lepton icon for header image

OS

Windows 10

OS - Others

No response

Firefox Version

Nightly (64-bit)

Distribution

Theme

Theme - More Info

No response

user.js setup

user.js setup
```javascript // I can't upload the user.js setup data since my Nightly instance was on Windows Sandbox VM. ```

Additional context

No response

Aleksandra0056 commented 2 weeks ago

This also applies to the tab title icon: This also applies to the tab title icon

Aleksandra0056 commented 2 weeks ago

Two thick old profiling icons are also located in Process Manager:

One is on the Process Manager tab title: Process Manager tab title icon

The other is on "Profile all threads of this process for 5 seconds" item: The profile all threads of this process for 5 seconds uses old icon

Aleksandra0056 commented 2 weeks ago

Thank you, it's now fixed.