The remote debugger "Connecting..." string and animation should be larger

NEW
Unassigned

Status

()

Firefox
Developer Tools: Framework
--
enhancement
5 years ago
5 years ago

People

(Reporter: Ioana (away), Unassigned)

Tracking

(Blocks: 1 bug, {regression})

21 Branch
regression
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

5 years ago
STR:
1. Enable remote debugging.
2. Go to Tools -> Developer Tools and click "Connect".
3. Click on the "Connect" button.

The "Connecting..." string and animation displayed at this point are much too small considering all the empty space and the much larger fonts around it.

Notes:
1. Reproducible on the latest Fx21 beta, Fx22 aurora, and Fx23 nightly builds. 
2. The string and animation are larger on Firefox 20, but they could be enlarged even more than there.
3. This is most probably a regression from bug 826685.
(Reporter)

Updated

5 years ago
Keywords: regression
(Reporter)

Updated

5 years ago
Blocks: 858765
Could you provide a screenshot for reference?
(Reporter)

Comment 2

5 years ago
Created attachment 739438 [details]
Firefox 20 screenshot
(Reporter)

Comment 3

5 years ago
Created attachment 739439 [details]
Firefox 21 screenshot
Based on the screenshots it seems like both the title "Connect to remote device" and the throbber text "Connecting..." were scaled down equally. It would appear that the throbber text was scaled down to match the footer text and the title text was scaled down by equal proportions. This make me think the change was intentional.

Looking at your screenshots the text is not unreadable or uncomfortable to read, at least to me. Unless this was an unintentional regression I'm inclined to resolve this invalid. I personally think the Firefox 21 screenshot is an improvement over the Firefox 20 screenshot.

That said, if you feel this warrants resolution we could move this to an enhancement request.
(Reporter)

Comment 5

5 years ago
I think you should consider bug 839515 and the reasoning there before changing this one. It is exactly the same situation, they regressed at the same point because of the same bug.

This bug can also be considered part of bug 858765, which is a meta bug and not an enhancement.
(In reply to Ioana Budnar [QA] from comment #5)
> I think you should consider bug 839515 and the reasoning there before
> changing this one. It is exactly the same situation, they regressed at the
> same point because of the same bug.

The improvement I made in bug 839515 felt adequate in my (subjective) opinion. The font is still smaller than before, but not unreadable, at least for me (and if you have seen my glasses you'll know that I'm not your average user :-))
 
> This bug can also be considered part of bug 858765, which is a meta bug and
> not an enhancement.

Maybe, even though the changes planned in the dependent bugs seem more like technical refactorings. In any case we would either need UX buy-in to change the font size or discuss it with paul who made the original change (intentionally I believe). FYI, Paul is on PTO for the next month or so.
(Reporter)

Comment 7

5 years ago
Thanks for your input Panos.

Marking the bug as an enhancement request considering comment 6.
Severity: normal → enhancement
You need to log in before you can comment on or make changes to this bug.