Commit Graph

54 Commits

Author SHA1 Message Date
Nikki Sharpley
434c90c533 Bug 1928116 - Update search synced tabs string for sidebar and firefox view r=sidebar-reviewers,fluent-reviewers,flod,fxview-reviewers,jsudiaman
Differential Revision: https://phabricator.services.mozilla.com/D228235
2024-11-08 15:28:03 +00:00
Florian Quèze
03b00e2a53 Bug 1918702 - Migrate telemetry events to use Glean APIs for firefoxview, r=chutten,fxview-reviewers,jsudiaman.
Differential Revision: https://phabricator.services.mozilla.com/D222416
2024-09-20 13:58:15 +00:00
Jonathan Sudiaman
da4a683e56 Bug 1861400 - Remove sync if disabled by policy r=fxview-reviewers,nsharpley,kcochrane
Brings the Firefox View behavior in line with what is done in `browser-sync.js`.

Differential Revision: https://phabricator.services.mozilla.com/D205881
2024-04-01 21:50:10 +00:00
Kelly Cochrane
f9d8aa7f85 Bug 1883850 - Ensure selected moz-page-nav button remains selected upon page refresh of Fx View r=fxview-reviewers,jsudiaman
Differential Revision: https://phabricator.services.mozilla.com/D203783
2024-03-06 21:13:29 +00:00
Jonathan Sudiaman
79a9016c9f Bug 1883235 - [Firefox View] The search box cannot fully display CJK characters r=fxview-reviewers,sclements
[Input size](https://searchfox.org/mozilla-central/source/browser/components/firefoxview/opentabs.mjs#216) of `30` approximately matches what's shown in [the spec](https://www.figma.com/file/wP9MY9KJn9xqPf140lTvTv/Fx-View-Specs-2023?type=design&node-id=6730-179702&mode=design&t=9DlykASuAzE3hP9S-0). Establish that as the minimum, and it will automatically expand for other locales as needed.

Differential Revision: https://phabricator.services.mozilla.com/D203477
2024-03-04 20:33:13 +00:00
Kelly Cochrane
d5647a2076 Bug 1850591 - Convert Firefox View's domain-specific navigation component to a global component r=desktop-theme-reviewers,fxview-reviewers,reusable-components-reviewers,sclements,jules,hjones,fluent-reviewers,flod
Differential Revision: https://phabricator.services.mozilla.com/D193593
2024-02-20 18:58:04 +00:00
Noemi Erli
8534246c91 Backed out changeset f19610ecbade (bug 1850591) for causing failures in browser_opentabs_tab_indicators.js CLOSED TREE 2024-02-10 01:07:49 +02:00
Kelly Cochrane
6b7daf459f Bug 1850591 - Convert Firefox View's domain-specific navigation component to a global component r=desktop-theme-reviewers,fxview-reviewers,reusable-components-reviewers,sclements,jules,hjones,fluent-reviewers,flod
Differential Revision: https://phabricator.services.mozilla.com/D193593
2024-02-09 22:01:30 +00:00
Butkovits Atila
1b1bf5d507 Backed out changeset e97e98c4b7c2 (bug 1850591) for causing failures at browser_1856572_ensure_Fluent_works_in_customizeMode.js. CLOSED TREE 2024-02-09 17:05:14 +02:00
Kelly Cochrane
9e6e12965a Bug 1850591 - Convert Firefox View's domain-specific navigation component to a global component r=desktop-theme-reviewers,fxview-reviewers,reusable-components-reviewers,sclements,jules,hjones,fluent-reviewers,flod
Differential Revision: https://phabricator.services.mozilla.com/D193593
2024-02-09 13:43:18 +00:00
Nikki Sharpley
17c37d523d Bug 1869836 - Rename files with Firefox View Next to Firefox View. r=fxview-reviewers,sfoster
- rename files to exclude "next" suffix
- move tests out of firefoxview-next folder into parent folder
- updat AboutRedirector to point at firefoxview.html
- this patch also resolves Bug 1872919 by adding a scroll into view during the `search_synced_tabs_recent_browsing` test

Differential Revision: https://phabricator.services.mozilla.com/D196347
2024-01-18 14:32:33 +00:00
Cristian Tuns
7368009cd2 Backed out changeset 7ec1dad4bca4 (bug 1869836) for causing bc failures in browser_syncedtabs_firefoxview.js CLOSED TREE 2024-01-16 17:36:03 -05:00
Nikki Sharpley
ac2d496df5 Bug 1869836 - Rename files with Firefox View Next to Firefox View. r=fxview-reviewers,sfoster
- rename files to exclude "next" suffix
- move tests out of firefoxview-next folder into parent folder
- updat AboutRedirector to point at firefoxview.html

Differential Revision: https://phabricator.services.mozilla.com/D196347
2024-01-16 21:46:47 +00:00
Norisz Fay
920554c646 Backed out changeset 1ec7a77be890 (bug 1869836) for causing bc and lint failures CLOSED TREE 2024-01-15 19:09:33 +02:00
Nikki Sharpley
9e4b7d5342 Bug 1869836 - Rename files with Firefox View Next to Firefox View. r=fxview-reviewers,sfoster
- rename files to exclude "next" suffix
- move tests out of firefoxview-next folder into parent folder
- updat AboutRedirector to point at firefoxview.html

Differential Revision: https://phabricator.services.mozilla.com/D196347
2024-01-15 15:34:34 +00:00
Nikki Sharpley
ca1dc84fa7 Bug 1869833 - Remove all legacy Firefox View files and assets. r=sfoster,fxview-reviewers,extension-reviewers
- removes all old View related files and assets
- removes legacy telemetry
- removes tests related to prefs that have since been removed
- remove `browser_all_files_referenced.js` exceptions added in Bug 1869833

Depends on D196093

Differential Revision: https://phabricator.services.mozilla.com/D196208
2024-01-09 20:52:58 +00:00
Stanca Serban
20610cad88 Backed out changeset 7eda7968bffa (bug 1869833) for causing mochitests failures in browser_parsable_css.js and in browser_firefoxview_next_general_telemetry.js. CLOSED TREE 2024-01-04 21:17:36 +02:00
Nikki Sharpley
0bcd4efffc Bug 1869833 - Remove all legacy Firefox View files and assets. r=sfoster,fxview-reviewers
- removes all old View related files and assets
- removes legacy telemetry
- removes tests related to prefs that have since been removed
- remove `browser_all_files_referenced.js` exceptions added in Bug 1869833

Differential Revision: https://phabricator.services.mozilla.com/D196208
2024-01-04 17:37:01 +00:00
Sam Foster
01194f3438 Bug 1852622 - Add a test-only firefoxview-entered notification, and modify the test helpers to use it. r=sclements,fxview-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D189443
2023-10-05 23:44:56 +00:00
Kelly Cochrane
5c6f7639fa Bug 1852989 - Add telemetry probe for Firefox View tab clicks r=sclements,fxview-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D188854
2023-09-29 18:49:12 +00:00
Sandor Molnar
1571f9b604 Backed out changeset 3220838c5990 (bug 1852989) for causing bc failures on browser/components/firefoxview/tests/browser/browser_recently_closed_tabs.js CLOSED TREE 2023-09-28 21:11:30 +03:00
Kelly Cochrane
5da1b9a7c0 Bug 1852989 - Add telemetry probe for Firefox View tab clicks r=sclements,fxview-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D188854
2023-09-28 13:36:16 +00:00
Shane Hughes
99e82b0f08 Bug 1821826 - Refactor FeatureCallout to support generic triggers. r=omc-reviewers,fxview-reviewers,tabbrowser-reviewers,dao,jprickett,sclements
Also disable the Firefox View feature tour to avoid any risk of
regressions. The feature tour code will be removed in a later patch.
It's still present for now for testing purposes.

Differential Revision: https://phabricator.services.mozilla.com/D180927
2023-07-11 16:00:40 +00:00
Shane Hughes
7cf9a86c67 Bug 1818829 - Implement Feature Callout theme configuration. r=jprickett,omc-reviewers,fxview-reviewers,sfoster
Add logic to apply theme colors to Feature Callout based on where it's
going to show. We can use in-content CSS properties for Firefox View and
other themed system pages, but not for PDF.js, nor for any callouts we
might show in the browser chrome in the future. For the browser chrome
in general, we can use the lightweight theme properties directly, in the
same way the chrome frontend does. But PDF.js is a special case, since
although it exists in the chrome, it's meant to appear like it's in the
PDF.js viewer. And the PDF.js viewer has its own theme totally
independent of everything else. So this dynamically applies themes from
different sources.

This also fixes the bug where the PDF.js color scheme could mismatch the
PDF.js viewer if the browser theme and system color scheme don't match,
e.g. where system color scheme is light but a dark theme is installed,
or vice versa. For PDF.js specifically, we can use the
-moz-content-prefers-color-scheme media query to follow the color scheme
as it exists in the PDF.js viewer page instead of the color scheme in
the chrome window where the Feature Callout actually exists.

It also adds or modifies some colors that were previously missing or
different from the prototype, fixes the illegibility of buttons in HCM
and forced colors mode, and makes some other minor color changes.

Differential Revision: https://phabricator.services.mozilla.com/D173088
2023-03-28 00:38:42 +00:00
Cosmin Sabou
c31085cb42 Backed out changeset fecef27ba1fa (bug 1818829) for causing newtab failures on _feature-callout-theme.scss. CLOSED TREE 2023-03-28 02:46:44 +03:00
Shane Hughes
43bb7f08b0 Bug 1818829 - Implement Feature Callout theme configuration. r=jprickett,omc-reviewers,fxview-reviewers,sfoster
Add logic to apply theme colors to Feature Callout based on where it's
going to show. We can use in-content CSS properties for Firefox View and
other themed system pages, but not for PDF.js, nor for any callouts we
might show in the browser chrome in the future. For the browser chrome
in general, we can use the lightweight theme properties directly, in the
same way the chrome frontend does. But PDF.js is a special case, since
although it exists in the chrome, it's meant to appear like it's in the
PDF.js viewer. And the PDF.js viewer has its own theme totally
independent of everything else. So this dynamically applies themes from
different sources.

This also fixes the bug where the PDF.js color scheme could mismatch the
PDF.js viewer if the browser theme and system color scheme don't match,
e.g. where system color scheme is light but a dark theme is installed,
or vice versa. For PDF.js specifically, we can use the
-moz-content-prefers-color-scheme media query to follow the color scheme
as it exists in the PDF.js viewer page instead of the color scheme in
the chrome window where the Feature Callout actually exists.

It also adds or modifies some colors that were previously missing or
different from the prototype, fixes the illegibility of buttons in HCM
and forced colors mode, and makes some other minor color changes.

Differential Revision: https://phabricator.services.mozilla.com/D173088
2023-03-27 23:07:45 +00:00
Dão Gottwald
1057069b47 Bug 1820475 - Remove colorways-card.mjs and related code. r=Gijs
Differential Revision: https://phabricator.services.mozilla.com/D171696
2023-03-07 12:35:52 +00:00
Csoregi Natalia
0350da18d5 Backed out changeset d59fe9707337 (bug 1820475) for failures on browser_multistage_spotlight.js. CLOSED TREE 2023-03-07 12:35:33 +02:00
Dão Gottwald
4bf57c85ea Bug 1820475 - Remove colorways-card.mjs and related code. r=Gijs
Differential Revision: https://phabricator.services.mozilla.com/D171696
2023-03-07 09:24:21 +00:00
Meg Viar
04e1d3b419 Bug 1806425 - Make value of source in Feature Callout and page in its telementry more consistent r=jprickett,aminomancer
Unify the values of "source" and "page" as used in FeatureCallout.sys.mjs:
- Explicitly pass in a value for "page" when instantiating a Feature Callout and use this for the value of "page" when sending Feature Callout telemetry and as the "source" when making calls to `sendTriggerMessage`. This avoids the risk of including non-about: page URLs or PDF file extensions in our telemetry.
- Set the value of "page" in an HTML data attribute that can be accessed for use in about:welcome telemetry for Spotlight and Feature Callouts.
- Update references to the page value previously used as the page/source for telemetry from `about:firefoxview` Feature Callouts from "firefoxview"  to "about:firefoxview"
- Pass the token "chrome" when creating a callout from the browser chrome and update references to the source in PDF.js messages' targeting
- Update the page value expected in automated tests as needed

Differential Revision: https://phabricator.services.mozilla.com/D165910
2023-01-20 16:53:30 +00:00
Meg Viar
79eec33848 Bug 1805930 - Change featureCallout.mjs to featureCallout.sys.mjs and update all references to filename r=barret,aminomancer
Differential Revision: https://phabricator.services.mozilla.com/D164824
2022-12-29 14:27:02 +00:00
Dão Gottwald
cfc7cc8b18 Bug 1801037 - Remove "New colorways are on the way" message from Firefox View. r=sfoster
Differential Revision: https://phabricator.services.mozilla.com/D162264
2022-11-18 06:25:41 +00:00
Tim Giles
9fa180f7e9 Bug 1794474 - Force tabs sync when a user reloads Firefox View. r=sfoster
Differential Revision: https://phabricator.services.mozilla.com/D159401
2022-10-17 18:39:27 +00:00
Bernard Igiri
bec652a910 Bug 1780239 - Fix screen reader order of Colorway Landmark in FxView r=Gijs
Added dynamic DOM reordering to FX View so that the order of announcement in screen readers and tab sorting, matches the visual changes made by the responsive CSS layout.

Differential Revision: https://phabricator.services.mozilla.com/D157256
2022-10-07 20:21:49 +00:00
Mark Banner
79b9557a71 Bug 1792398 - Enable ESLint rule 'strict' on mjs files as the directive is not necessary for modules. r=arai,pip-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D158115
2022-09-26 21:47:50 +00:00
Marian-Vasile Laza
b7f6194b8d Backed out changeset 0679274d6ed5 (bug 1792398) for causing bc failures on browser_sendQuery.js. CLOSED TREE 2022-09-26 22:53:00 +03:00
Mark Banner
5dcae1ed44 Bug 1792398 - Enable ESLint rule 'strict' on mjs files as the directive is not necessary for modules. r=arai,pip-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D158115
2022-09-26 18:51:57 +00:00
Narcis Beleuzu
95ae6b7d0f Backed out changeset 849faf753208 (bug 1792398) for bc failures on browser_sendQuery.js . CLOSED TREE 2022-09-26 20:44:40 +03:00
Mark Banner
ff212fab8b Bug 1792398 - Enable ESLint rule 'strict' on mjs files as the directive is not necessary for modules. r=arai,pip-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D158115
2022-09-26 16:51:10 +00:00
Katherine Patenio
377e9d145e Bug 1747229 - implement telemetry probes for colorway closet r=bigiri,dao,rpl
Differential Revision: https://phabricator.services.mozilla.com/D151771
2022-08-26 17:05:58 +00:00
Meg Viar
6358232229 Bug 1781084 - Add CSS transitions when moving Feature Callout from one element to the next r=Mardak
Depends on D154143

Differential Revision: https://phabricator.services.mozilla.com/D154275
2022-08-15 23:11:13 +00:00
Niklas Baumgardner
ba5d19f1f6 Bug 1779642 - Add telemetry for Firefox View. r=Gijs,sfoster
Differential Revision: https://phabricator.services.mozilla.com/D152437
2022-08-09 12:47:38 +00:00
Meg Viar
1717fd75c8 Bug 1779024 - Create Feature Callout message surface using aboutwelcome r=pdahiya,Gijs,Mardak
Differential Revision: https://phabricator.services.mozilla.com/D152449
2022-08-04 11:47:01 +00:00
Sam Foster
617a3d5b9c Bug 1780168 - Rename TabsPickupContainer to TabPickupContainer for consistency. r=sclements
Differential Revision: https://phabricator.services.mozilla.com/D152220
2022-07-21 22:47:35 +00:00
Sam Foster
e0acecab85 Bug 1780168 - Extract tabsSetupFlowManager to be a system ESM. r=Gijs
* Move the tabsSetupFlowManager into its own system ESM. This allows multiple about:firefoxview instances to have a single manager and syncronize state without duplication
* The first call to tabSetupFlowManager.initialize() will do any necessary setup to wait for sync & fxa services to be ready
* tabSetupFlowManager handles its own uninit
* Move TabsPickupContainer to match file naming convention for our other custom element modules
* TabsPickupContainer instances observe tabSetupFlowManager's notifications for state changes, and can request the current uiStateIndex directly.
* The utility openFxaSignup and openSyncPreferences functions now need know which window to open the URL in

Differential Revision: https://phabricator.services.mozilla.com/D152151
2022-07-21 22:47:35 +00:00
Bernard Igiri
cbd7df1750 Bug 1776221 - Added auto update of Colorway CTA on theme change r=sfoster
With this fix changing the Colorway theme should automatically update the Colorway CTA in Firefox View. Additionally, the Colorways portion of firefoxview.mjs has been migrated to colorways-card.mjs and restructured to match the style of the rest of the Firefox View codebase.

Differential Revision: https://phabricator.services.mozilla.com/D150878
2022-07-15 21:36:56 +00:00
Norisz Fay
d9002d885b Backed out changeset 4ad72b0c451f (bug 1776221) for causing bc failures on browser_colorways_card.js CLOSED TREE 2022-07-13 20:17:42 +03:00
Bernard Igiri
cb1b4cc87f Bug 1776221 - Added auto update of Colorway CTA on theme change r=dao
With this fix changing the Colorway theme should automatically update the Colorway CTA in Firefox View. Additionally, the Colorways portion of firefoxview.mjs has been migrated to colorways-card.mjs and restructured to match the style of the rest of the Firefox View codebase.

Differential Revision: https://phabricator.services.mozilla.com/D150878
2022-07-13 14:12:04 +00:00
Tooru Fujisawa
328d63ce7f Bug 1777486 - Part 2: Migrate XPCOMUtils.jsm consumers with automatic migration. r=webdriver-reviewers,geckoview-reviewers,preferences-reviewers,application-update-reviewers,pip-reviewers,kmag,owlish,whimboo
Differential Revision: https://phabricator.services.mozilla.com/D151214
2022-07-12 04:21:34 +00:00
Andreea Pavel
f009ba2983 Backed out 11 changesets (Bug 1777486) for failing bc at browser_startup.js on a CLOSED TREE
Backed out changeset b6c4c386f1a6 (Bug 1777486)
Backed out changeset 195cc2de8433 (Bug 1777486)
Backed out changeset 20c746fb1648 (Bug 1777486)
Backed out changeset d5fd8173d62d (Bug 1777486)
Backed out changeset 6d758fab5a3e (Bug 1777486)
Backed out changeset e938b601ba15 (Bug 1777486)
Backed out changeset 0c4ea0b9416b (Bug 1777486)
Backed out changeset 0559c53cc668 (Bug 1777486)
Backed out changeset eea573d3a9f9 (Bug 1777486)
Backed out changeset 9ce3a6496a49 (bug 1777486)
Backed out changeset b0867652fc48 (bug 1777486)
2022-07-11 22:24:40 +03:00