Closed Bug 1078455 Opened 5 years ago Closed 5 years ago
UITour: Add ability to highlight Dev
. Tools and Web IDE toolbar buttons
Add ability to highlight Dev Tools in chrome. Use case at hand: In the /firstrun experience for the developer edition browser, we will: - highlight icon in toolbar (appears as default in toolbar for this build) - extend a doorhanger from the feature icon with a brief message describing feature. Doorhanger will also include 'not now' link and green CTA button from our standard UITour doorhanger template. We will not open the feature itself for this /firstrun tour. Possible future use cases could be highlighting feature from a snippet, product page, or future first-run tour.
Added the ID's to the UITour's target, and modified the unit tests too to work with both m-c and gum. This patch fixes also bug 1078454. Matt, I set you as reviewer because you reviewed a similar patch for the loop button; but feel free to assign to anyone else if you think he/her fits more. Thanks!
Attachment #8512086 - Flags: review?(MattN+bmo)
Comment on attachment 8512086 [details] [diff] [review] highlight-buttons.patch Review of attachment 8512086 [details] [diff] [review]: ----------------------------------------------------------------- LGTM
Attachment #8512086 - Flags: review?(MattN+bmo) → review+
Duplicate of this bug: 1078454
Summary: UITour: Add ability to highlight Dev Tools in toolbar → UITour: Add ability to highlight Dev. Tools and WebIDE toolbar buttons
OS: Mac OS X → All
Hardware: x86 → All
jwalker asked me to push this to try, land it in fx-team and request Aurora approval. Try: https://tbpl.mozilla.org/?tree=Try&rev=dbcd50b2e929 https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=dbcd50b2e929
Comment on attachment 8512086 [details] [diff] [review] highlight-buttons.patch Approval Request Comment [Feature/regressing bug #]:UI Tour [User impact if declined]:No ability to highlight DevTools buttons in the UI tour [Describe test coverage new/current, TBPL]: green on try [Risks and why]: Very low risk [String/UUID change made/needed]:None
Attachment #8512086 - Flags: approval-mozilla-aurora?
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 36
Attachment #8512086 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
https://hg.mozilla.org/releases/mozilla-aurora/rev/caaf56e6e947 (In reply to Michael Ratcliffe [:miker] [:mratcliffe] from comment #5) > https://hg.mozilla.org/integration/fx-team/rev/ea1cb099c537 In the future, you should probably be more careful about double-checking the commit information before pushing on somebody else's behalf. And Matteo, you should probably make sure you have Mercurial properly configured to generate patches with the correct metadata included from the start :) https://developer.mozilla.org/en-US/docs/Mercurial_FAQ#How_can_I_generate_a_patch_for_somebody_else_to_check-in_for_me.3F
When will we be able to view this in Gum?
It is present on Gum now.
(In reply to Panos Astithas [:past] (overloaded, please needinfo) from comment #10) > It is present on Gum now. I don't see either "devtools" or "webide" when querying getConfiguration('availableTargets') in 35.0a2 (2014-11-03). Can someone please provide a link to the latest build so I can check again?
I can see this on gum: gum$ hg log -r caaf56e6e947 changeset: 225809:caaf56e6e947 user: Matteo Ferretti <firstname.lastname@example.org> date: Thu Oct 30 13:35:25 2014 +0000 summary: Bug 1078455 - UITour: Add ability to highlight Dev. Tools and WebIDE toolbar buttons. r=MattN+bmo, a=lsblakk I don't know why it doesn't seem to work though. You can find the latest builds at: https://treeherder.mozilla.org/ui/#/jobs?repo=gum
I downloaded gum from the link provided by Panos, and I checked that that build has the patch: jar:file:///Users/ZER0/Desktop/FirefoxDeveloperEditionDebug.app/Contents/Resources/browser/omni.ja!/modules/UITour.jsm The patch also modified the unit test, that use `getConfiguration('availableTargets')`, so I'm pretty sure the method returns both `devtools` and `webide` (the last one if the pref is set): http://mxr.mozilla.org/mozilla-central/source/browser/modules/test/browser_UITour_availableTargets.js#28 So I guess it should works with this build, Alex. But let me know if there are any issue.
Thanks Matteo, When I download the latest build from http://ftp.mozilla.org/pub/mozilla.org/firefox/tinderbox-builds/gum-macosx64/latest/ I see the targets as expected! Looks like this patch just landed :) Oddly, the latest build comes up as Aurora for me (with the light UI and no dev branding). Not sure why, but that's unrelated to this bug.
(In reply to Alex Gibson [:agibson] from comment #14) > When I download the latest build from > http://ftp.mozilla.org/pub/mozilla.org/firefox/tinderbox-builds/gum-macosx64/ > latest/ I see the targets as expected! Looks like this patch just landed :) Unfortunately that URL does not contain the actual latest build (perhaps because there are some jobs still pending). You should download the one here: http://ftp.mozilla.org/pub/mozilla.org/firefox/tinderbox-builds/gum-macosx64/1415065657/ > Oddly, the latest build comes up as Aurora for me (with the light UI and no > dev branding). Not sure why, but that's unrelated to this bug. This is because that build didn't have all the necessary changes.
You need to log in before you can comment on or make changes to this bug.