Commit Graph

123 Commits

Author SHA1 Message Date
Cosmin Sabou
5623f8ad8f Backed out changeset f7e1c167b99b (bug 1915007) for causing bc failures on browser_view_sidebar_menu. CLOSED TREE 2024-10-09 18:30:34 +03:00
Nikki Sharpley
92fbf7e3c4 Bug 1915007 - Avoid duplicate View>Sidebar menu items on sidebar pref flip r=sidebar-reviewers,Gijs
Differential Revision: https://phabricator.services.mozilla.com/D224493
2024-10-09 13:36:22 +00:00
Kelly Cochrane
a38bb565f9 Bug 1908989 - Add animations to sidebar launcher in 'Always hide' mode r=sidebar-reviewers,emilio,sfoster
Differential Revision: https://phabricator.services.mozilla.com/D220485
2024-10-08 16:49:41 +00:00
Gijs Kruitbosch
e68340d490 Bug 1923326 - fix toolbarbutton access on startup when sidebar button isn't in the DOM yet, r=sidebar-reviewers,jsudiaman,nsharpley
Differential Revision: https://phabricator.services.mozilla.com/D224930
2024-10-08 13:43:41 +00:00
Sam Foster
4c7602730b Bug 1914359 - Ensure new sidebar launcher shows when toggling the pref while a legacy sidebar is open. r=sidebar-reviewers,Gijs
Differential Revision: https://phabricator.services.mozilla.com/D224406
2024-10-03 20:22:16 +00:00
Nikki Sharpley
b9c3ba706c Bug 1914452 - Update revamp sidebar button to reflect sidebar states r=sidebar-reviewers,desktop-theme-reviewers,dao,sfoster,jsudiaman
- different sidebar button icons for expanded, collapsed sidebar states
- move sidebar-button to beginning of navbar strip, to the left of back button
- update tests to reflect new sidebar button positioning
- add hover animation to button

Differential Revision: https://phabricator.services.mozilla.com/D223406
2024-10-03 14:20:55 +00:00
Sandor Molnar
fbc2bb1bd2 Backed out changeset 6ad99c7c787d (bug 1914452) for causing bc sider bar related failures. CLOSED TREE 2024-10-02 21:25:44 +03:00
Nikki Sharpley
20a153c56b Bug 1914452 - Update revamp sidebar button to reflect sidebar states r=sidebar-reviewers,desktop-theme-reviewers,dao,sfoster,jsudiaman
- different sidebar button icons for expanded, collapsed and hidden sidebar states
- move sidebar-button to beginning of navbar strip, to the left of back button
- update tests to reflect new sidebar button positioning

Differential Revision: https://phabricator.services.mozilla.com/D223406
2024-10-02 16:52:46 +00:00
Jonathan Sudiaman
40b2800c49 Bug 1921539 - Sidebar button has no label when moved to the overflow menu r=sidebar-reviewers,fluent-reviewers,Gijs,sclements
Fluent messages were missing `label` attribute. Use the existing one from customizableWidgets.properties.

Differential Revision: https://phabricator.services.mozilla.com/D224167
2024-10-01 14:30:04 +00:00
Iulian Moraru
7f4f522517 Backed out changeset 5808b8ac42ad (bug 1914452) for causing multiple bc failures. CLOSED TREE 2024-09-28 02:36:30 +03:00
Iulian Moraru
d2bc47834b Backed out changeset 1b56a9e3db55 (bug 1921562) for causing conflicts when trying to backout Bug 1914452. 2024-09-28 02:35:55 +03:00
Nikki Sharpley
28adb5f06a Bug 1921562 - Update and animate sidebar toolbar button states r=sidebar-reviewers,desktop-theme-reviewers,sfoster
- follow up to correct patch for Bug 1914452 and add animations

Differential Revision: https://phabricator.services.mozilla.com/D224025
2024-09-27 22:16:32 +00:00
Nikki Sharpley
8536a92e8d Bug 1914452 - Update revamp sidebar button to reflect sidebar states r=sidebar-reviewers,desktop-theme-reviewers,sfoster,jsudiaman
- different sidebar button icons for expanded, collapsed and hidden sidebar states
- move sidebar-button to beginning of navbar strip, to the left of back button

Differential Revision: https://phabricator.services.mozilla.com/D223406
2024-09-27 20:33:59 +00:00
Jonathan Sudiaman
03df9fb463 Bug 1902056 - Update strings for Sidebar toolbarbutton, close button and Customize Sidebar visibility settings r=sidebar-reviewers,fluent-reviewers,sclements,bolsson
Differential Revision: https://phabricator.services.mozilla.com/D223314
2024-09-25 23:30:38 +00:00
Sandor Molnar
7599f7c3a6 Backed out changeset ef1b42c36db7 (bug 1902056) for causing perma bc failures @ browser_toolbarKeyNav.js CLOSED TREE 2024-09-25 14:07:40 +03:00
Fred Chasen
a31102ffc2 Bug 1919166 - Prevent registering review checker sidebar panel for private windows. r=shopping-reviewers,sidebar-reviewers,kpatenio,nsharpley
Differential Revision: https://phabricator.services.mozilla.com/D222380
2024-09-24 22:00:18 +00:00
Jonathan Sudiaman
375579d22d Bug 1902056 - Update strings for Sidebar toolbarbutton, close button and Customize Sidebar visibility settings r=sidebar-reviewers,fluent-reviewers,sclements,bolsson
Differential Revision: https://phabricator.services.mozilla.com/D223314
2024-09-24 20:00:26 +00:00
Kelly Cochrane
8cacd1c229 Bug 1906888 - Update focus order and move new tab button to the bottom for vertical tabs r=desktop-theme-reviewers,tabbrowser-reviewers,dao,sidebar-reviewers,ayeddi
Differential Revision: https://phabricator.services.mozilla.com/D221244
2024-09-20 11:40:04 +00:00
Fred Chasen
89e6d0b8cb Bug 1916017 - Add Review Checker as a sidebar tool. r=shopping-reviewers,fluent-reviewers,desktop-theme-reviewers,sidebar-reviewers,sclements,kpatenio
- Registers the a new review checker sidebar using `registerPrefSidebar` that will display if  `browser.shopping.experience2023.integratedSidebar` is enabled.
- Hides any open review checker sidebars if the pref changes.
- Adds `reviewchecker` to the sidebar revamp tools pref.

Note: this only adds the sidebar tool, it will not load anything as the actors aren't setup yet.

Differential Revision: https://phabricator.services.mozilla.com/D221362
2024-09-19 17:07:29 +00:00
Butkovits Atila
5a9a789f82 Backed out changeset fb2f7f9f6def (bug 1906888) for causing failures at browser_newtabButton.js. CLOSED TREE 2024-09-19 22:04:46 +03:00
Kelly Cochrane
a83714bc93 Bug 1906888 - Update focus order and move new tab button to the bottom for vertical tabs r=desktop-theme-reviewers,tabbrowser-reviewers,dao,sidebar-reviewers,ayeddi
Differential Revision: https://phabricator.services.mozilla.com/D221244
2024-09-19 15:54:02 +00:00
Emilio Cobos Álvarez
de32a21400 Bug 1919041 - Handle interrupting sidebar animation. r=sidebar-reviewers,kcochrane,sclements
Differential Revision: https://phabricator.services.mozilla.com/D222297
2024-09-17 17:03:39 +00:00
Dão Gottwald
23da2b3201 Bug 1918676 - Disable tab opening / closing animations and tab size locking in vertical mode, properly set --tab-min-width rather than overriding min-width in vertical mode, and restrict some styles to horizontal mode. r=sidebar-reviewers,desktop-theme-reviewers,tabbrowser-reviewers,emilio,kcochrane,sclements,dwalker
Differential Revision: https://phabricator.services.mozilla.com/D222202
2024-09-17 08:21:24 +00:00
Noemi Erli
353976cead Backed out 2 changesets (bug 1918676, bug 1918638) for causing browser chrome failures
Backed out changeset c36ab1469893 (bug 1918638)
Backed out changeset f973faab6674 (bug 1918676)
2024-09-17 02:34:41 +03:00
Dão Gottwald
2b53d56022 Bug 1918676 - Disable tab opening / closing animations and tab size locking in vertical mode, properly set --tab-min-width rather than overriding min-width in vertical mode, and restrict some styles to horizontal mode. r=sidebar-reviewers,desktop-theme-reviewers,tabbrowser-reviewers,emilio,kcochrane,sclements,dwalker
Differential Revision: https://phabricator.services.mozilla.com/D222202
2024-09-16 19:53:42 +00:00
Jonathan Sudiaman
15ad15eecf Bug 1898250 - Add telemetry to sidebar r=sidebar-reviewers,fxview-reviewers,sclements
Differential Revision: https://phabricator.services.mozilla.com/D220720
2024-09-16 15:42:48 +00:00
Emilio Cobos Álvarez
2b868a8902 Bug 1918765 - Use ease-in-out easing for sidebar animation. r=sidebar-reviewers,kcochrane
Restores the easing from before the new animation work.

Differential Revision: https://phabricator.services.mozilla.com/D222185
2024-09-14 23:13:39 +00:00
Kelly Cochrane
16d633fe66 Bug 1911889 - Update tab mute/soundplaying/activemedia-blocked icon styling r=desktop-theme-reviewers,tabbrowser-reviewers,sidebar-reviewers,nsharpley,dao,sclements
Differential Revision: https://phabricator.services.mozilla.com/D219495
2024-09-13 17:51:03 +00:00
Iulian Moraru
2e1e9ca43c Backed out changeset e6b82ff4e092 (bug 1911889) for causing bc failures on browser_tabIconOverlayPiP.js. CLOSED TREE 2024-09-13 19:14:13 +03:00
Kelly Cochrane
744bb885ad Bug 1911889 - Update tab mute/soundplaying/activemedia-blocked icon styling r=desktop-theme-reviewers,tabbrowser-reviewers,sidebar-reviewers,nsharpley,dao,sclements
Differential Revision: https://phabricator.services.mozilla.com/D219495
2024-09-13 14:46:04 +00:00
Emilio Cobos Álvarez
3e676b7e1d Bug 1915230 - Make animation and duration configurable. r=#sidebar-reviewers CLOSED TREE
And tweak the default duration as per UX feedback.

Differential Revision: https://phabricator.services.mozilla.com/D221718

Depends on D221351
2024-09-12 19:14:23 +00:00
Emilio Cobos Álvarez
6bb5f60b53 Bug 1915230 - Use a translate transform only. r=#sidebar-reviewers!,mstange
This:

 * Works around the WebRender issue from bug 1916599.
 * Might be a bit simpler to reason about (the sidebar special-cases to
   keep the buttons at the right edge are a bit unfortunate but...).
 * Avoids the squishiness that Markus and Sam were discussing in
   #gfx-firefox).

There's still some flickering on the <browser> element, but I have a
good idea of how to fix it (basically, we're triggering resizes from
layout, but for remote iframes we really don't want to do that, because
it can cause raciness / flickering as observed).

Differential Revision: https://phabricator.services.mozilla.com/D221351

Depends on D220846
2024-09-12 10:02:04 +00:00
Emilio Cobos Álvarez
39dbb62d25 Bug 1915230 - Make sidebar animation use transforms. r=#sidebar-reviewers!,mconley!,dao!
This looks simpler, a lot smoother on my machine, and shouldn't cause
content resizes. It also doesn't set custom properties on the root which
causes an expensive restyle, specially on big documents (like with many
tabs open).

Maybe eventually we can use view transitions for this, seems like a good
use-case.

We need to move the background to `#sidebar-main` rather than the
wrapper because during the shrinking animation the wrapper is small and
the `#sidebar-main` element would be transparent, which we don't want.

Differential Revision: https://phabricator.services.mozilla.com/D220846
2024-09-12 10:02:04 +00:00
Butkovits Atila
1a38648aa0 Backed out 3 changesets (bug 1915230) for causing failures at test_restore_sidebar.py. CLOSED TREE
Backed out changeset 05fe9b71651f (bug 1915230)
Backed out changeset 749011ce7a8a (bug 1915230)
Backed out changeset aaaae84198ae (bug 1915230)
2024-09-12 19:12:43 +03:00
Emilio Cobos Álvarez
b36505ece7 Bug 1915230 - Make animation and duration configurable. r=sidebar-reviewers,kcochrane
And tweak the default duration as per UX feedback.

Differential Revision: https://phabricator.services.mozilla.com/D221718
2024-09-12 10:02:05 +00:00
Emilio Cobos Álvarez
d9fad8f233 Bug 1915230 - Use a translate transform only. r=sidebar-reviewers,desktop-theme-reviewers,kcochrane,dao
This:

 * Works around the WebRender issue from bug 1916599.
 * Might be a bit simpler to reason about (the sidebar special-cases to
   keep the buttons at the right edge are a bit unfortunate but...).
 * Avoids the squishiness that Markus and Sam were discussing in
   #gfx-firefox).

There's still some flickering on the <browser> element, but I have a
good idea of how to fix it (basically, we're triggering resizes from
layout, but for remote iframes we really don't want to do that, because
it can cause raciness / flickering as observed).

Differential Revision: https://phabricator.services.mozilla.com/D221351
2024-09-12 10:02:04 +00:00
Emilio Cobos Álvarez
2fc55271bc Bug 1915230 - Make sidebar animation use transforms. r=mconley,dao,sidebar-reviewers,desktop-theme-reviewers,kcochrane
This looks simpler, a lot smoother on my machine, and shouldn't cause
content resizes. It also doesn't set custom properties on the root which
causes an expensive restyle, specially on big documents (like with many
tabs open).

Maybe eventually we can use view transitions for this, seems like a good
use-case.

We need to move the background to `#sidebar-main` rather than the
wrapper because during the shrinking animation the wrapper is small and
the `#sidebar-main` element would be transparent, which we don't want.

Differential Revision: https://phabricator.services.mozilla.com/D220846
2024-09-12 10:02:04 +00:00
Sam Foster
adb5a03beb Bug 1899346 - Move CUI widgets out of the horizontal tabstrip when in vertical tabs mode. r=sidebar-reviewers,mconley,sclements,webdriver-reviewers,whimboo,tabbrowser-reviewers
* Ensure tabstrip widgets are temporarily removeable at browser init so the CUI placements are correctly applied
* Shuffle placements during initialization to build the correct toolbars for the verticalTabs pref value
* Notify on the 'tabstrip-orientation-change' topic when the verticalTabs pref changes and CUI placements have been updated
* Add tests for switching tabstrip orientation, and for initializing in verticalTabs mode

Differential Revision: https://phabricator.services.mozilla.com/D217161
2024-09-11 16:27:08 +00:00
Micah Tigley
a22f833a97 Bug 1917709 - Rename megalist pref to contextual-password-manager. r=issammani,rsafaeian,firefox-desktop-core-reviewers ,sidebar-reviewers,dao,jsudiaman
Differential Revision: https://phabricator.services.mozilla.com/D221554
2024-09-11 16:02:02 +00:00
Cristina Horotan
211657609a Backed out changeset 0c7c121f1fa3 (bug 1899346) for causing Mn failures at test_quit_restart.py CLOSED TREE 2024-09-04 05:34:33 +03:00
Sam Foster
5ebd63c578 Bug 1899346 - Move CUI widgets out of the horizontal tabstrip when in vertical tabs mode. r=sidebar-reviewers,mconley,sclements,webdriver-reviewers,whimboo
* Ensure tabstrip widgets are temporarily removeable at browser init so the CUI placements are correctly applied
* Shuffle placements during initialization to build the correct toolbars for the verticalTabs pref value
* Notify on the 'tabstrip-orientation-change' topic when the verticalTabs pref changes and CUI placements have been updated
* Add tests for switching tabstrip orientation, and for initializing in verticalTabs mode

Differential Revision: https://phabricator.services.mozilla.com/D217161
2024-09-04 00:12:07 +00:00
Ed Lee
d1d69b2d3a Bug 1895932 - Add sidebar prefs to nimbus r=sidebar-reviewers,Gijs,sclements
Add new SidebarManager singleton with init on import that allows setting user branch prefs specified in the nimbus variable for sidebar.

Differential Revision: https://phabricator.services.mozilla.com/D219903
2024-09-02 22:43:28 +00:00
Emilio Cobos Álvarez
f7551929f8 Bug 1916098 - Remove appcontent box. r=dao,desktop-theme-reviewers,devtools-reviewers,tabbrowser-reviewers,nchevobbe
gNotificationBox doesn't go there anymore (it goes inside the toolbox),
so we have really no use for that.

Simplify how the devtools theme is set on the browser, so that it is set
on the root.

Differential Revision: https://phabricator.services.mozilla.com/D220769
2024-09-02 13:47:45 +00:00
Kelly Cochrane
86f229c8a5 Bug 1915821 - Remove all screenshots after sidebar expand/collapse animation r=sidebar-reviewers,nsharpley
Differential Revision: https://phabricator.services.mozilla.com/D220736
2024-08-31 13:16:48 +00:00
Kelly Cochrane
fceef0f8b6 Bug 1915232 - Fix for sidebar toolbar button inverted state r=sidebar-reviewers,nsharpley
Differential Revision: https://phabricator.services.mozilla.com/D220735
2024-08-30 19:45:30 +00:00
Stanca Serban
37b5e2bd39 Backed out changeset f20f138ea4ac (bug 1915232) for causing mochitests failures in browser_history_sidebar.js. CLOSED TREE 2024-08-31 03:31:19 +03:00
Stanca Serban
1466a02554 Backed out changeset 8b97e9376724 (bug 1915821) for causing OS X mochitests failures in browser_toolbarButtonKeyPress.js. CLOSED TREE 2024-08-31 02:32:35 +03:00
Kelly Cochrane
0372b26dea Bug 1915232 - Fix for sidebar toolbar button inverted state r=sidebar-reviewers,nsharpley
Differential Revision: https://phabricator.services.mozilla.com/D220735
2024-08-30 19:45:30 +00:00
Kelly Cochrane
1e5899abe1 Bug 1915821 - Remove all screenshots after sidebar expand/collapse animation r=sidebar-reviewers,nsharpley
Differential Revision: https://phabricator.services.mozilla.com/D220736
2024-08-30 19:45:09 +00:00
Ed Lee
857840a797 Bug 1915335 - Display sidebar tool pref changes in existing windows r=sidebar-reviewers,jsudiaman
Detect pref change to refresh disabled state to update if any changed.

Differential Revision: https://phabricator.services.mozilla.com/D220370
2024-08-29 00:52:13 +00:00