Commit graph

52 commits

Author SHA1 Message Date
Jonathan Sudiaman
e883ef8dfe 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
9dc1679874 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
fc2c64faf7 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
0a51eb37a7 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
426e311669 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
fa0e1f01ae 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
86107ae8e9 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
d72308b393 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
57db0ea127 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
014080f990 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
80020bf88f 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
ae3df68e9b Backed out changeset 1ec7a77be890 (bug 1869836) for causing bc and lint failures CLOSED TREE 2024-01-15 19:09:33 +02:00
Nikki Sharpley
ea95272239 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
f26d7471aa 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
4e1cdb227e 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
ebc4c40237 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
3d1fa88e86 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
9a6092b559 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
067ffdbbc7 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
b81d391f3f 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
9c7a1ae2e1 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
5d946f3d90 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
948fdb0dfb 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
effb48b0f5 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
3c21bc3bef 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
422f2645bd 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
438ea15da5 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
a3e99a2057 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
90822c532c 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
4d433a2196 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
cb9c26280e 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
d4ae4b52b6 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
a9ba6e830d 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
130adc5d9a 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
3e99025d45 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
6c2d0d8cea 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
05dcda5a7f 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
ac33ac38e2 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
3dfe381d04 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
31dcd4d2eb 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
eadd77632f 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
e5aff81dc0 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
ade2730960 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
4b6eaac0e3 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
41d7ccc557 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
d830629ca2 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
54623364da 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
7738a75fdd 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
Tooru Fujisawa
8cd6ed7409 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-11 15:09:13 +00:00
Bernard Igiri
f2a37376ae Bug 1776682 - Localize Colorway CTA in Firefox View r=dao,fluent-reviewers,flod
Applied Colorway localization from Bug 1770030 to Colorway CTA in Firefox View.

Differential Revision: https://phabricator.services.mozilla.com/D150862
2022-07-07 12:30:17 +00:00