Closed Bug 1113299 Opened 5 years ago Closed 5 years ago

Hide "Mirror Tab" entry instead of disabling it when no mirroring devices are available

Categories

(Firefox :: General, defect)

defect
Not set
Points:
1

Tracking

()

RESOLVED FIXED
Firefox 38
Iteration:
38.1 - 26 Jan
Tracking Status
firefox36 --- wontfix
firefox37 --- fixed
firefox38 --- fixed

People

(Reporter: Gijs, Assigned: Gijs)

References

Details

Attachments

(1 file)

As per summary. I don't think this should be always-visible, considering most users don't actually have a device with which they can mirror. We can show them if/when they do.
Attachment #8540149 - Flags: review?(jaws)
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Iteration: --- → 37.2
Points: --- → 1
Flags: qe-verify-
Flags: in-testsuite-
Flags: firefox-backlog+
Iteration: 37.2 → 37.3
Comment on attachment 8540149 [details] [diff] [review]
hide tab mirroring feature if unavailable,

Review of attachment 8540149 [details] [diff] [review]:
-----------------------------------------------------------------

A disabled menuitem wasn't any good for people finding out how to start using the feature. With that said, how are people supposed to find out about the feature and also how they can use it? The alternative would be to always enable the menuitem and have a sub-menuitem that when clicked goes to a Getting Started/Tour URL.
Attachment #8540149 - Flags: review?(jaws) → review+
Comment on attachment 8540149 [details] [diff] [review]
hide tab mirroring feature if unavailable,

Asking for ui-review per comment 2
Attachment #8540149 - Flags: ui-review?(philipp)
(In reply to (Limited availability until January 6) Jared Wein [:jaws] (please needinfo? me) from comment #2)
> A disabled menuitem wasn't any good for people finding out how to start
> using the feature. With that said, how are people supposed to find out about
> the feature and also how they can use it?

I don't think we need to worry about that at this point.

> The alternative would be to always
> enable the menuitem and have a sub-menuitem that when clicked goes to a
> Getting Started/Tour URL.

Maybe later, but for now this seems like the best solution.
Iteration: 37.3 - 12 Jan → 38.1 - 26 Jan
Gijs, based on Gavin's response let's go ahead and land this now and file a bug for any necessary changes once we get feedback from Philipp.
Flags: needinfo?(gijskruitbosch+bugs)
OK:

remote:   https://hg.mozilla.org/integration/fx-team/rev/dfc93c68f9c7


By now this is on 37, and this will hit 38, so we should uplift this to Aurora, I guess. :-(
Flags: needinfo?(gijskruitbosch+bugs)
Comment on attachment 8540149 [details] [diff] [review]
hide tab mirroring feature if unavailable,

Approval Request Comment
[Feature/regressing bug #]: bug 1088758
[User impact if declined]: always-disabled menuitem in the tools menu
[Describe test coverage new/current, TBPL]: nope
[Risks and why]: very low, switching it from disabled to hidden
[String/UUID change made/needed]: no
Attachment #8540149 - Flags: approval-mozilla-aurora?
(In reply to :Gijs Kruitbosch from comment #7)
> [User impact if declined]: always-disabled menuitem in the tools menu

Sorry, this should have finished:

... for most users
https://hg.mozilla.org/mozilla-central/rev/dfc93c68f9c7
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 38
Comment on attachment 8540149 [details] [diff] [review]
hide tab mirroring feature if unavailable,

Clearing the ui-review since this has already landed.
I didn't test the patch itself, but from the description and discussion it sounds fine.
Attachment #8540149 - Flags: ui-review?(philipp)
Comment on attachment 8540149 [details] [diff] [review]
hide tab mirroring feature if unavailable,

Aurora+
Attachment #8540149 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.