Closed Bug 1474095 Opened 2 years ago Closed 2 years ago

Addon-tip-icon for web-ext in about:debugging cut (at low window width)

Categories

(DevTools :: about:debugging, defect)

defect
Not set
normal

Tracking

(firefox61 wontfix, firefox62 wontfix, firefox63 fixed)

RESOLVED FIXED
Firefox 63
Tracking Status
firefox61 --- wontfix
firefox62 --- wontfix
firefox63 --- fixed

People

(Reporter: c4609174, Assigned: c4609174)

Details

Attachments

(1 file, 1 obsolete file)

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

Steps to reproduce:

1. Set the language of your Firefox to German.
2. Go to about:debugging.


Actual results:

The "? icon" there is cut. 

See https://hostux.pics/images/2018/07/07/d75022fbef53e82932e1643a1dc8dee3.png


Expected results:

It should keep it's size.

**I would like to work on this bug and actually I've already found a solution.**
(Also reproducible in English if you make the window width quite low…)
I managed to reproduce this issue on Firefox 61.0.1, 62.0b7 and Nightly 63.0a1 (2018-07-11), Ubuntu 16.04.
Status: UNCONFIRMED → NEW
Component: Untriaged → Graphics
Ever confirmed: true
Product: Firefox → Core
Version: 61 Branch → Trunk
Great, so I'd like to take this, as mentioned before.

I'd just get my invitation for https://janitor.technology/, so I can start with it.
Thanks to Janitor.technology, so I could made this. Works really great!
Attachment #8991463 - Flags: review?(nical.bugzilla)
Anyone here? I just clicked some review buttons… don't know how to actually get one to review this? Is there anything I can do?
rguk, based on the file that you have modified, this bug is in the wrong component. I'm going to move it to the DevTools component so that the Dev Tools developers can find this. You will likely have better luck finding an appropriate reviewer there.

:Harald Kirschner, are you able to review this patch or redirect to someone more appropriate?
Assignee: nobody → c4609174
Component: Graphics → about:debugging
Flags: needinfo?(hkirschner)
Product: Core → DevTools
Comment on attachment 8991463 [details] [diff] [review]
Prevent shrinking of the add-on tip icon in about:debugging

Jan, can you have a look? Thanks.
Attachment #8991463 - Flags: review?(nical.bugzilla) → review?(janx)
Comment on attachment 8991463 [details] [diff] [review]
Prevent shrinking of the add-on tip icon in about:debugging

Hi rugk, I was able to confirm that your patch indeed fixes the bug. Thank you for working on this! (Also, I'm glad that janitor.technology was useful to you!)

As a previous owner of about:debugging, I approve this fix. Please just update the patch description to our usual format, like so:

> Bug 1474095 - Prevent shrinking of the add-on tip icon in about:debugging. r=janx

Then, please feel free to set the "checkin-needed" keyword on this bug (if you can't or don't know how, please feel free to "needinfo?" me).
Flags: needinfo?(hkirschner)
Attachment #8991463 - Flags: review?(janx) → review+
Thanks Jan for jumping in!
changed description as per request of @janx
Attachment #8991463 - Attachment is obsolete: true
Attachment #8997388 - Flags: checkin+
Keywords: checkin-needed
Pushed by ncsoregi@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e6a6fd029201
"Addon-tip-icon for web-ext in about:debugging cut (at low window width)" [c=c4609174]. r=janx
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/e6a6fd029201
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
rugk, congratulations on your first patch to Firefox! (Belatedly, and if I'm not mistaken.)

As you've probably seen in the tracking flags, your improvement to about:debugging will be live in Firefox 63. :)
Ah, yeah, thanks. (And thanks for making Janitor.technology, yet again. :) )


Unless I forget it, I'll certainly celebrate it when FF 63 get's stable. ;)

BTW if anyone could vouch for me on Mozillians, I am https://mozillians.org/rugk there.

You need to log in before you can comment on or make changes to this bug.