44 bytes, text/x-github-pull-request
|Details | Review | Splinter Review|
Implement custom site search, indexing numerous Mozilla websites, to evaluate its effectiveness and usefulness. The test will entail three Google custom search engines: 1) Activist - focused on advocacy and educational content. 2) Developer - focused on technical content. 3) General - broad search across all (or almost all) Mozilla websites. We'll target these searches on a few key pages on mozilla.org. Shared drive with lots of docs and info: https://drive.google.com/drive/u/1/folders/0B5fr2uXG8zVqcllibURNV3d2T2M
Design project folder for cross site search: https://drive.google.com/drive/folders/0Bw8GuIJD9hBMT1RNQlRoY21GV2c?usp=sharing Sketch File for cross site search options on dev pages: https://drive.google.com/a/mozilla.com/file/d/0Bw8GuIJD9hBMaWREcnR0SVR5YUE/view?usp=sharing
For those w/out Sketch here are links to review placement options for search bar on developer focused pages of Mozilla.org PNGs in this folder: https://drive.google.com/a/mozilla.com/file/d/0Bw8GuIJD9hBMaWREcnR0SVR5YUE/view?usp=sharing A multi-page PDF: https://drive.google.com/a/mozilla.com/file/d/0Bw8GuIJD9hBMZnN6dWhucl9WOTg/view?usp=sharing
(In reply to Michael Ham from comment #2) > For those w/out Sketch here are links to review placement options for search > bar on developer focused pages of Mozilla.org > > PNGs in this folder: > https://drive.google.com/a/mozilla.com/file/d/0Bw8GuIJD9hBMaWREcnR0SVR5YUE/ > view?usp=sharing > > A multi-page PDF: > https://drive.google.com/a/mozilla.com/file/d/0Bw8GuIJD9hBMZnN6dWhucl9WOTg/ > view?usp=sharing Michael and I have reviewed together: Channel page - use mock up #1 /technology - use mock up #1 /firefox/developer - use mock up #2
Hi all – Below are cross site search options for the /contribute and 404 pages. Sketch File for cross site search options on general pages: https://drive.google.com/a/mozilla.com/file/d/0Bw8GuIJD9hBMaW1zVTNSYWJOZ28/view?usp=sharing PNGs of cross site search on general pages for review: https://drive.google.com/drive/folders/0Bw8GuIJD9hBMNjJVSGVvX1hNQU0?usp=sharing
404 I like. Contribute, my first thought was that it was too prominent. I'd be curious what Jenn thinks about the placement.
(In reply to Ben (:bensternthal) from comment #5) > 404 I like. > Contribute, my first thought was that it was too prominent. I'd be curious > what Jenn thinks about the placement. Thanks for the feedback, Ben. Yeah – let's look at what search looks like under the nav on /contribute. I'll post that shortly.
Commits pushed to master at https://github.com/mozilla/bedrock https://github.com/mozilla/bedrock/commit/44c996595076b5517840a6b0a22c30f5513fbc4f [bug 1319244] test Activist cross-site search https://github.com/mozilla/bedrock/commit/b1df264b39252df67b2c65deccecc54fc9addc66 Merge pull request #4498 from craigcook/bug-1319244-test-cross-site-search [bug 1319244] test Activist cross-site search
Here's another take on the search bar for the /contribute page. The search bar tucks under the nav and the page indicator (triangle) drops down underneath the search bar. /Contribute Search Option 2: https://drive.google.com/open?id=0Bw8GuIJD9hBMT3I0UUNDaFUzVGc
Commits pushed to master at https://github.com/mozilla/bedrock https://github.com/mozilla/bedrock/commit/68cf669561b4cf93396e38fda19967a197e0d223 [bug 1319244] Remove activist search traffic cop Increasing the test audience to 100% of English visitors https://github.com/mozilla/bedrock/commit/5eb72609383418ab596ff367f332c6bdd7e944cb Merge pull request #4517 from craigcook/bug-1319244-remove-cop-activist-search [bug 1319244] Remove activist search traffic cop
Commits pushed to master at https://github.com/mozilla/bedrock https://github.com/mozilla/bedrock/commit/9b36263747bba3951c5ad8d1041e7d73616c78d2 [bug 1319244] Test developer and general search * Convert search include to macro. * Use traffic cop for Firefox pages. https://github.com/mozilla/bedrock/commit/a99d303ec1b90bc43a5c78a0f395c107c99698cb Merge pull request #4519 from craigcook/bug-1319244-test-developer-search [bug 1319244] Test developer and general search
Commits pushed to master at https://github.com/mozilla/bedrock https://github.com/mozilla/bedrock/commit/860c845a7568f7f3c308af0d8cc12cf3ef9186a0 [bug 1319244] Fix firefox/developer URL https://github.com/mozilla/bedrock/commit/39e0429f25926d1de844206161bd0ceab48accee Merge pull request #4524 from craigcook/bug-1319244-test-developer-search [bug 1319244] Fix firefox/developer URL
Created attachment 8831906 [details] [review] Github pull request https://github.com/mozilla/bedrock/pull/4615 PR to remove all the search code now that the test is complete.
Assignee: nobody → craigcook.bugz
Commits pushed to master at https://github.com/mozilla/bedrock https://github.com/mozilla/bedrock/commit/cde59b40737c7c85a5329cf1d6ec3567227b217f [fix bug 1319244] Remove search test code https://github.com/mozilla/bedrock/commit/e9559d6e303c65401f032fcd238d42508cb48c06 Merge pull request #4615 from craigcook/bug-1319244-remove-search-test-code [fix bug 1319244] Remove search test code
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.