Skip to content

Commit

Permalink
Move toolbar import to WebUIs that need it.
Browse files Browse the repository at this point in the history
Removes br_toolbar from polymer_overriding and imports it only in UIs
that need it.
  • Loading branch information
mkarolin committed Nov 15, 2024
1 parent a3ad284 commit 5c41833
Show file tree
Hide file tree
Showing 6 changed files with 10 additions and 1 deletion.
2 changes: 2 additions & 0 deletions browser/resources/bookmarks/brave_overrides/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,6 @@
// License, v. 2.0. If a copy of the MPL was not distributed with this file,
// you can obtain one at https://mozilla.org/MPL/2.0/.

import 'chrome://resources/brave/br_elements/br_toolbar/br_toolbar.js'

import './folder_node.js'
2 changes: 2 additions & 0 deletions browser/resources/extensions/brave_overrides/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@
// License, v. 2.0. If a copy of the MPL was not distributed with this file,
// you can obtain one at https://mozilla.org/MPL/2.0/.

import 'chrome://resources/brave/br_elements/br_toolbar/br_toolbar.js'

import './extensions_detail_view.js'
import './extensions_toolbar.js'
import './side_bar.js'
2 changes: 2 additions & 0 deletions browser/resources/history/brave_overrides/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@
// License, v. 2.0. If a copy of the MPL was not distributed with this file,
// you can obtain one at https://mozilla.org/MPL/2.0/.

import 'chrome://resources/brave/br_elements/br_toolbar/br_toolbar.js'

import './app.js'
import './history_item.js'
import './side_bar.js'
2 changes: 2 additions & 0 deletions browser/resources/password_manager/brave_overrides/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,6 @@
// License, v. 2.0. If a copy of the MPL was not distributed with this file,
// You can obtain one at https://mozilla.org/MPL/2.0/.

import 'chrome://resources/brave/br_elements/br_toolbar/br_toolbar.js'

import './side_bar.js'
2 changes: 2 additions & 0 deletions browser/resources/settings/brave_overrides/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,8 @@
// where it is not consistant due to "network" responses) and
// optimized (rollup controls the order in which modules are executed).

import 'chrome://resources/brave/br_elements/br_toolbar/br_toolbar.js'

import './config.js'
import { ContentSettingsTypes } from '../site_settings/constants.js'
import './about_page.js'
Expand Down
1 change: 0 additions & 1 deletion ui/webui/resources/polymer_overriding.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@
// @ts-nocheck TODO(petemill): Define types

// Global overrides
import './br_elements/br_toolbar/br_toolbar.js'
import './br_elements/br_shared_style.css.js'

import { html as polymerHtml, mixinBehaviors, Polymer, PolymerElement } from 'chrome://resources/polymer/v3_0/polymer/polymer_bundled.min.js';
Expand Down

0 comments on commit 5c41833

Please sign in to comment.