Text wrapping issue in the detail page of the extension

VERIFIED FIXED in Firefox 66

Status

()

defect
P5
normal
VERIFIED FIXED
3 months ago
2 months ago

People

(Reporter: cbadescu, Assigned: championshuttler, Mentored)

Tracking

({good-first-bug})

Trunk
mozilla67
Points:
---

Firefox Tracking Flags

(firefox64 wontfix, firefox65 wontfix, firefox66 verified, firefox67 verified)

Details

Attachments

(5 attachments)

(Reporter)

Description

3 months ago
Posted image Bug.png

[Affected versions]:

  • Firefox 66.0a1 (20190111093148)
  • Firefox 65.0b9 (20190107180200)
  • Firefox 64.0.2 (20190108160530)

[Affected platforms]:

  • Win 7 64-bit
  • Mac OS X 10.13.3

[Steps to reproduce]:
1.Install https://addons.mozilla.org/en-US/firefox/addon/dark-background-light-text/
2.Navigate to Extensions from about:addons page.
3.Click on the extension to see the detail page.
4.Observe the description section.

[Expected results]:

  • The text is wrapped correctly in the description section.

[Actual results]:

  • A small portion of the text has wrapping issues in the description section.

Please see the attached screenshot.

Priority: -- → P5

Adding word-wrap: break-word; should to the #detail-fulldesc element should fix this.

Mentor: mstriemer

This could be a good first bug.

Keywords: good-first-bug

Comment 3

3 months ago

I am a first time contributor, and I see how to fix this bug, but where exactly to I make the pull request for the source code from? That is such a newbie question but any help is appreciated.

Updated

3 months ago
Flags: needinfo?(mstriemer)
(Assignee)

Comment 4

2 months ago

Hi Ryan , are you working on it? If not can i take this up? I build the firefox and setup mercurial as well.

Thanks

Flags: needinfo?(ryanwade1928)

Comment 5

2 months ago

Sure you can take it up Shivam. I have not been able to get a working build of firefox yet. Thanks for checking!

Comment 6

2 months ago

(In reply to Shivam Singhal [ :championshuttler ] from comment #4)

Hi Ryan , are you working on it? If not can i take this up? I build the firefox and setup mercurial as well.

Thanks

Sure you can take it up Shivam. I have not been able to get a working build of firefox yet. Thanks for checking!

Flags: needinfo?(ryanwade1928)
(Assignee)

Updated

2 months ago
Assignee: nobody → shivams2799
Posted image After.png
Posted image Before.png

Sorry I missed your needinfo request, Ryan.

There is documentation for getting Firefox setup [1], you can use an artifact build for changes like this. Once you have a patch you will need to submit it to phabricator for review. There is documentation [2] for that too, I think you'll need to complete the steps up to "Submitting Patches" and you'll probably want to use arc for now. I don't have any experience with moz-phab but you can give it a try if you'd like.

Let me know if there's anything else I can help with!

[1] https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Build_Instructions/Simple_Firefox_build
[2] https://moz-conduit.readthedocs.io/en/latest/phabricator-user.html

Flags: needinfo?(mstriemer)

Comment 11

2 months ago
Pushed by mstriemer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1818b96796ae
Fixes text wrapping issues in extension detail page. r=mstriemer,Gijs

Comment 12

2 months ago
bugherder
Status: NEW → RESOLVED
Last Resolved: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
(Reporter)

Comment 13

2 months ago
Posted image Bug1519376.png

This issue is verified as fixed on Firefox 67.0a1 (20190212095015) under Win 7 64-bit and Mac OS X 10.14.1.

Please see the attached screenshot.

(Reporter)

Updated

2 months ago
Status: RESOLVED → VERIFIED

Hi, I'm happy to take this fix in beta 66 if you can request beta uplift.

Comment on attachment 9042764 [details]
Bug 1519376 - Fixes text wrapping issues in extension detail page. r=mstriemer

Beta/Release Uplift Approval Request

Feature/Bug causing the regression

None

User impact if declined

Long unbroken lines will extend beyond the card in about:addons.

Is this code covered by automated tests?

No

Has the fix been verified in Nightly?

Yes

Needs manual test from QE?

No

If yes, steps to reproduce

List of other uplifts needed

None

Risk to taking this patch

Low

Why is the change risky/not risky? (and alternatives if risky)

Simple one-line CSS change

String changes made/needed

Attachment #9042764 - Flags: approval-mozilla-beta?
Flags: needinfo?(shivams2799)

Comment on attachment 9042764 [details]
Bug 1519376 - Fixes text wrapping issues in extension detail page. r=mstriemer

CSS fix, ok for beta 10.

Attachment #9042764 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
(Reporter)

Comment 18

2 months ago

This issue is verified as fixed on Firefox 66.0b10 (20190221160854) under Win 7 64-bit and Mac OS X 10.14.1.

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