1378108 - Remove Firebug theme

Status: RESOLVED FIXED

(DevTools :: Framework, enhancement, P3)

Description

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Firebug theme landed in devtools primarily to help with Firebug -> DevTools transition. 

Since all add-ons has been deprecated (since Firefox 57) and replaced by WebExtensions, it isn't possible to install Firebug at all and thus there is no transition path. The additional (transition) UI we introduced in Firebug isn't available anymore.

Also, the Firebug theme was introduced for Firefox Australis theme (~2012) and might feel a bit old these days. It's probably good time to remove the theme entirely.

Honza

Comment 1

[Patrick Brosset <:pbro>]

As discussed last week, and in a meeting this week, I agree with this. Once we start shipping a new, Photon-compatible, design, I don't think we should be maintaining the Firebug theme anymore.
We might take this on during Q3, within the Perf & Polish project that Mike and Gabriel are working on (CC'ing them now).

Comment 2

[Patrick Brosset <:pbro>]

Before rushing into this, we implemented a new probe to gather actual data (bug 1396811).
Here's data from this probe in Nightly 58: https://telemetry.mozilla.org/new-pipeline/dist.html#!cumulative=1&end_date=2017-09-27&keys=LIGHT!DARK!FIREBUG!__none__&max_channel_version=nightly%252F58&measure=SCALARS_DEVTOOLS.CURRENT_THEME&min_channel_version=null&processType=*&product=Firefox&sanitize=1&sort_keys=submissions&start_date=2017-09-21&table=0&trim=1&use_submission_date=0

The numbers are: 18.5k light theme, 8.2k dark theme, 375 Firebug. This means 1.4% for Firebug only.
Now, we need to wait until we get numbers on release 56 too, only there will we know what our users actually use and what the ratio of Firebug theme users is.

But so far, the numbers are telling us that the amount of time we spend maintaining the Firebug theme is probably not worth it.
Recently, a lot of time was put in bug 1389730 to refresh our light and dark themes. Doing this in a way that the Firebug theme still worked proved very difficult. So this theme has an important maintenance cost.

Comment 3

[Patrick Brosset <:pbro>]

We now have more numbers on release 56.
I created a spreadsheet with the data I found relevant in it: https://docs.google.com/spreadsheets/d/1haDZMiHpYRbwGR2mgvVmUlCOUZFmGLwN90KE4ARXbzI/edit#gid=0

Comment 4

[Tim Nguyen :ntim]

This is now actionable, so marking as good-first-bug.

https://github.com/devtools-html/rfcs/issues/42

Comment 5

[Brian Grinstead [:bgrins]]

I don't think this is ready to marked as a good first bug, at least pending a decision on what the migration plan will be.

Comment 6

[Brian Grinstead [:bgrins]]

By the way, I have a WIP patch that removes most of the rules at https://github.com/devtools-html/rfcs/issues/42#issuecomment-369675504. It could be landed separately to begin simplifying our CSS, or could be modified to remove the theme entirely.

Comment 7

[Brian Grinstead [:bgrins]]

Attachment: Bug 1378108 - Part 1 - Remove non-variable css selector overrides for Firebug theme;

Review commit: https://reviewboard.mozilla.org/r/233034/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/233034/

Comment 8

[Patrick Brosset <:pbro>]

(In reply to Brian Grinstead [:bgrins] from comment #5)
> I don't think this is ready to marked as a good first bug, at least pending
> a decision on what the migration plan will be.
I have captured the product requirements in this PRD: https://docs.google.com/document/d/1BjXRV8OwWf7V2Km6UPDNpcVBdvXcGaOO7rCVFAJzmiE/edit and got it reviewed by Harald, who agrees we do not require a migration plan.
Harald: can you provide a confirmation here please, so we can go ahead with this bug?

Comment 9

[Tim Nguyen :ntim]

The doc is private.

Comment 10

[:Harald Kirschner :digitarald]

(In reply to Tim Nguyen :ntim from comment #9)
> The doc is private.

Made available.

> Harald: can you provide a confirmation here please, so we can go ahead with this bug?

Green light from me.

Comment 11

[Brian Grinstead [:bgrins]]

Attachment: Bug 1378108 - Part 2 - Remove Firebug theme;

Review commit: https://reviewboard.mozilla.org/r/233740/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/233740/

Comment 12

[Brian Grinstead [:bgrins]]

Attachment: Bug 1378108 - Part 3 - Migrate Firebug theme users to the light theme;

Review commit: https://reviewboard.mozilla.org/r/233742/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/233742/

Comment 13

[Brian Grinstead [:bgrins]]

Comment on attachment 8964312 [details]
Bug 1378108 - Part 1 - Remove non-variable css selector overrides for Firebug theme;

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/233034/diff/1-2/

Comment 14

[Brian Grinstead [:bgrins]]

Comment on attachment 8964312 [details]
Bug 1378108 - Part 1 - Remove non-variable css selector overrides for Firebug theme;

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/233034/diff/2-3/

Comment 15

[Brian Grinstead [:bgrins]]

Comment on attachment 8964994 [details]
Bug 1378108 - Part 2 - Remove Firebug theme;

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/233740/diff/1-2/

Comment 16

[Brian Grinstead [:bgrins]]

Comment on attachment 8964995 [details]
Bug 1378108 - Part 3 - Migrate Firebug theme users to the light theme;

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/233742/diff/1-2/

Comment 17

[Gabriel Luong [:gl] (ΦωΦ)]

Comment on attachment 8964312 [details]
Bug 1378108 - Part 1 - Remove non-variable css selector overrides for Firebug theme;

https://reviewboard.mozilla.org/r/233034/#review239412

::: devtools/client/shared/components/tabs/Tabs.css:81
(Diff revision 3)
>    position: relative;
>    border-bottom: 1px solid var(--theme-splitter-color);
>    background: var(--theme-tab-toolbar-background);
>  }
>  
>  .theme-dark .tabs .tabs-menu-item,

Maybe unrelated and can be done seperately in another bug or patch, but it seems like we can remove theme-dark/light from these selectors since they were there to guard against applying the rules for the firebug theme.

::: devtools/client/shared/components/tree/TreeView.css:184
(Diff revision 3)
>  /******************************************************************************/
>  /* Themes */
>  
>  /* :not(.selected) is used because row selection styles should have
>     more precedence than row hovering. */
>  .theme-light .treeTable .treeRow:not(.selected):hover,

Same as above.

Comment 18

[Gabriel Luong [:gl] (ΦωΦ)]

Comment on attachment 8964994 [details]
Bug 1378108 - Part 2 - Remove Firebug theme;

https://reviewboard.mozilla.org/r/233740/#review239420

Comment 19

[:Gijs (he/him)]

Comment on attachment 8964995 [details]
Bug 1378108 - Part 3 - Migrate Firebug theme users to the light theme;

https://reviewboard.mozilla.org/r/233742/#review239490

Comment 20

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/058d1b523272
Part 1 - Remove non-variable css selector overrides for Firebug theme;r=gl
https://hg.mozilla.org/integration/autoland/rev/77eaf99d1363
Part 2 - Remove Firebug theme;r=gl
https://hg.mozilla.org/integration/autoland/rev/311320b0570a
Part 3 - Migrate Firebug theme users to the light theme;r=Gijs

Comment 21

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/058d1b523272
https://hg.mozilla.org/mozilla-central/rev/77eaf99d1363
https://hg.mozilla.org/mozilla-central/rev/311320b0570a

Comment 22

[Mohammad Maruf Rahman]

I have reproduced this bug with Nightly 56.0a1 (2017-07-05) on Ubuntu 16.04 LTS!

This bug's fix is Verified with latest Nightly !

Build   ID - 20180424220100
User Agent - Mozilla/5.0 (X11; Linux x86_64; rv:61.0) Gecko/20100101 Firefox/61.0

Comment 23

[:Harald Kirschner :digitarald]

Noticed that https://developer.mozilla.org/en-US/docs/Tools/Settings needs to be updated.

Comment 24

[Irene Smith]

Removed the reference to the Firebug theme from the page and replaced the image of Developer Tools Settings so that it no longer lists the Firebug theme as a choice.