1384983 - Manage Share Providers icon is missing on Mac

Status: RESOLVED FIXED

(Firefox :: Theme, defect, P1)

Description

[Dão Gottwald [:dao]]

+++ This bug was initially created as a clone of Bug #1384895 +++

(In reply to Johann Hofmann [:johannh] from bug 1384895 comment #3)
> This works for me but I should mention that on OSX the button is just blank,
> before and after this patch.

Comment 1

[Dão Gottwald [:dao]]

Probably just need to remove this rule:

http://searchfox.org/mozilla-central/rev/ad093e98f42338effe2e2513e26c3a311dd96422/browser/themes/osx/browser.css#1066-1069

Comment 2

[hanbin.chang]

Hi, I am new to the community and I would like to work on this as my first bug.
I have the test/dev environment set up on the virtual machine.
Could you please give me some advice on how I should test this?
Regards,

Comment 3

[Dão Gottwald [:dao]]

Attachment: expected Manage Share Provider icon

(In reply to hanbin.chang from comment #2)
> Hi, I am new to the community and I would like to work on this as my first
> bug.
> I have the test/dev environment set up on the virtual machine.
> Could you please give me some advice on how I should test this?
> Regards,

Hi. To test this, first put the share widget on the toolbar. The panel that opens from the button should have a gear icon in the top left corner.

Comment 4

[hanbin.chang]

Oh okay, I was wondering since the problem only occurs in Mac, I wasn't sure how to test this.
My virtual environment runs on Linux, so I am not sure how to reproduce this on Linux.
Also, could you assign this to me if possible? Regards

Comment 5

[Dão Gottwald [:dao]]

Yeah, you won't be able to see the problem on Linux. Do you want to work on this regardless?

Comment 6

[hanbin.chang]

It appears to be a simple fix if I am not underestimating it, and I would like to still work on this upon some guidance and mentoring(bugfix is part of credit for my software engineering course). I do have a Mac machine and I would guess that I need to set up the testing environment on my Mac or VM that has OSX?

Regards,

Comment 7

[Dão Gottwald [:dao]]

(In reply to hanbin.chang from comment #6)
> It appears to be a simple fix if I am not underestimating it, and I would
> like to still work on this upon some guidance and mentoring(bugfix is part
> of credit for my software engineering course).

How far along are you? Have you built Firefox? Have you modified browser/themes/osx/browser.css as needed?

> I do have a Mac machine and I
> would guess that I need to set up the testing environment on my Mac or VM
> that has OSX?

I can get you a Mac build from automation once we have a patch.

Comment 8

[hanbin.chang]

Hi, I have removed the line that you mentioned in an earlier reply and committed to my local repository.

Comment 9

[hanbin.chang]

Attachment: Bug 1384983 patchfile

Comment 10

[Dão Gottwald [:dao]]

Comment on attachment 8892133 [details] [diff] [review]
Bug 1384983 patchfile

Looks good. Thanks!

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9f26b6d47e7f
Remove obsolete #manage-share-providers rule. r=dao

Comment 12

[hanbin.chang]

Hi, what is the next step?
I am not entirely sure of the bugzilla process. Should I push the changes from my local repository?
Regards,

Comment 13

[Dão Gottwald [:dao]]

(In reply to hanbin.chang from comment #12)
> Hi, what is the next step?
> I am not entirely sure of the bugzilla process. Should I push the changes
> from my local repository?
> Regards,

First contributors can't push to the integration repositories, see https://www.mozilla.org/en-US/about/governance/policies/commit/

I pushed the patch for you in comment 11. It should automatically be merged to mozilla-central within a day.

Comment 14

[hanbin.chang]

Oh okay, thanks.

Comment 15

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/9f26b6d47e7f