Closed Bug 1519376 Opened 2 years ago Closed 2 years ago

Text wrapping issue in the detail page of the extension

Categories

(Toolkit :: Add-ons Manager, defect, P5)

defect

Tracking

()

VERIFIED FIXED
mozilla67
Tracking Status
firefox64 --- wontfix
firefox65 --- wontfix
firefox66 --- verified
firefox67 --- verified

People

(Reporter: cbadescu, Assigned: championshuttler, Mentored)

Details

(Keywords: good-first-bug)

Attachments

(5 files)

Attached 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

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.

Flags: needinfo?(mstriemer)

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)

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

(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: nobody → shivams2799
Attached image After.png
Attached 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)
Pushed by mstriemer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1818b96796ae
Fixes text wrapping issues in extension detail page. r=mstriemer,Gijs
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
Attached 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.

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+

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.