The profiler added to Firefox Development Tools in bug 795268 is not themeable. Theme files should have been placed in /skin/: chrome://browser/content/profiler.css /browser/devtools/profiler/cleopatra/css/ * etc...
Created attachment 694270 [details] [diff] [review] WIP: Patch to rearaange the profiler theme files to brower/skin/devtools
Note, the other changes in the *.css files should be covered in bug 822110 (removal of the enmpty noise.png, as well as the -webkit- and -moz- lines)
Created attachment 694277 [details] [diff] [review] V2: correct more issues in .css files, and correctly include profiler.css
Component: Developer Tools → Developer Tools: Profiler
this is actually not a trivial thing to do currently. The Profiler UI aka Cleopatra is from an upstream project, notably: https://github.com/robcee/cleopatra As new features are added to the Gecko profiler, they're reflected in Cleopatra. If we want to continue updating from that, we'll need to update our in-tree version with changes from that repo. Splitting out the styling would break those updates currently. We may get to a point where we're hosting the definitive version of Cleopatra at which point we can do theme-specific things with it, but until that time, we're stuck with this UI.
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → WONTFIX
Comment on attachment 694277 [details] [diff] [review] V2: correct more issues in .css files, and correctly include profiler.css canceling review for now. also, I linked to my fork of the canonical repository which is located here: https://github.com/bgirard/cleopatra
Can I at least request that the stylesheets and images to be placed in /skin/?
Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---
Unfortunately the styling is going to be hard to separate. Benoit, Anton, what do you guys think about hosting the styling for cleo-light in mozilla-central?
Status: REOPENED → RESOLVED
Last Resolved: 5 years ago → 4 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 879008
You need to log in before you can comment on or make changes to this bug.