Bug 1650304 Comment 3 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Thanks for your interest.
Of course you can work on this!

I'd suggest that you first follow the tutorial steps in [1], by using an __artifact build__ when you get the question. Artifact builds make it a lot faster to build firefox when all you want is changing Javascript / CSS stuff, because this doesn't rebuild all the c++/rust code.

The devtools folks also wrote interesting doc in [2] about the process when contributing (how to build, how to refresh, etc).

[1] https://firefox-source-docs.mozilla.org/contributing/how_to_contribute_firefox.html
[2] https://firefox-source-docs.mozilla.org/devtools/index.html#contributing

In this case I think the fix will be in CSS, the CSS for this component is in [3]. I believe the solution will involve adding a "flex" property somewhere. I suggest to look at the other checkboxes that are correct and you should find the solution.

[3] https://searchfox.org/mozilla-central/source/devtools/client/themes/aboutprofiling.css

Also, [4] is where the DOM is produced. I believe this is correct but I haven't looked closer.

[4] https://searchfox.org/mozilla-central/rev/692a10e624a02fe79bf45046b586e50ea0ff17f1/devtools/client/performance-new/components/Settings.js#315-333

Good luck hunting!
Thanks for your interest.
Of course you can work on this!

I'd suggest that you first follow the tutorial steps in [1], by using an __artifact build__ when you get the question. Artifact builds make it a lot faster to build firefox when all you want is changing Javascript / CSS stuff, because this doesn't rebuild all the c++/rust code.

The devtools folks also wrote interesting doc in [2] about the process when contributing (how to build, how to refresh, etc).

[1] https://firefox-source-docs.mozilla.org/contributing/how_to_contribute_firefox.html
[2] https://firefox-source-docs.mozilla.org/devtools/index.html#contributing

In this case I think the fix will be in CSS, the CSS for this component is in [3]. I believe the solution will involve adding a "flex" property somewhere. I suggest to look at the other checkboxes that are correct and you should find the solution.

[3] https://searchfox.org/mozilla-central/source/devtools/client/themes/aboutprofiling.css

Also, [4] is where the DOM is produced. I believe this is correct but I haven't looked closer.

[4] https://searchfox.org/mozilla-central/rev/692a10e624a02fe79bf45046b586e50ea0ff17f1/devtools/client/performance-new/components/Settings.js#315-333

Good luck hunting bugs!

Back to Bug 1650304 Comment 3