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

RESOLVED FIXED in Firefox 63

Status

defect
RESOLVED FIXED
11 months ago
2 months ago

People

(Reporter: c4609174, Assigned: c4609174)

Tracking

Trunk
Firefox 63

Firefox Tracking Flags

(firefox61 wontfix, firefox62 wontfix, firefox63 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Assignee

Description

11 months ago
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.**
Assignee

Comment 1

11 months ago
(Also reproducible in English if you make the window width quite low…)

Comment 2

11 months ago
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
Assignee

Comment 3

10 months ago
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.
Assignee

Comment 4

10 months ago
Thanks to Janitor.technology, so I could made this. Works really great!
Assignee

Updated

10 months ago
Attachment #8991463 - Flags: review?(nical.bugzilla)
Assignee

Comment 5

10 months ago
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 8

10 months ago
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!
Assignee

Comment 10

10 months ago
changed description as per request of @janx
Attachment #8991463 - Attachment is obsolete: true
Attachment #8997388 - Flags: checkin+
Assignee

Updated

10 months ago
Keywords: checkin-needed

Comment 11

10 months ago
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

Comment 12

10 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/e6a6fd029201
Status: NEW → RESOLVED
Last Resolved: 10 months 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. :)
Assignee

Comment 14

9 months ago
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. ;)
Assignee

Comment 15

2 months ago

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.