Skip to content
This repository was archived by the owner on Feb 17, 2025. It is now read-only.

Commit 34e1678

Browse files
authored
Bug 1535314 - Fix history panel sync icon color (#4632)
1 parent 2ecca21 commit 34e1678

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

Client/Frontend/Home/HistoryPanel.swift

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ private struct HistoryPanelUX {
1515
static let IconSize = 23
1616
static let IconBorderColor = UIColor.Photon.Grey30
1717
static let IconBorderWidth: CGFloat = 0.5
18+
static let actionIconColor = UIColor.Photon.Grey40 // Works for light and dark theme.
1819
}
1920

2021
private class FetchInProgressError: MaybeErrorType {
@@ -338,7 +339,7 @@ class HistoryPanel: SiteTableViewController, HomePanel {
338339
cell.textLabel?.text = Strings.HistoryPanelClearHistoryButtonTitle
339340
cell.detailTextLabel?.text = ""
340341
cell.imageView?.image = UIImage.templateImageNamed("forget")
341-
cell.imageView?.tintColor = UIColor(colorString: "0xb2b2b2")
342+
cell.imageView?.tintColor = HistoryPanelUX.actionIconColor
342343
cell.imageView?.backgroundColor = UIColor.theme.homePanel.historyHeaderIconsBackground
343344
cell.accessibilityIdentifier = "HistoryPanel.clearHistory"
344345

@@ -357,7 +358,8 @@ class HistoryPanel: SiteTableViewController, HomePanel {
357358
cell.accessoryType = .disclosureIndicator
358359
cell.textLabel?.text = Strings.RecentlyClosedTabsButtonTitle
359360
cell.detailTextLabel?.text = ""
360-
cell.imageView?.image = UIImage(named: "recently_closed")
361+
cell.imageView?.image = UIImage.templateImageNamed("recently_closed")
362+
cell.imageView?.tintColor = HistoryPanelUX.actionIconColor
361363
cell.imageView?.backgroundColor = UIColor.theme.homePanel.historyHeaderIconsBackground
362364
AdditionalHistoryActionRow.setStyle(enabled: hasRecentlyClosed, forCell: cell)
363365
cell.accessibilityIdentifier = "HistoryPanel.recentlyClosedCell"
@@ -368,7 +370,10 @@ class HistoryPanel: SiteTableViewController, HomePanel {
368370
cell.accessoryType = .disclosureIndicator
369371
cell.textLabel?.text = Strings.SyncedTabsTableViewCellTitle
370372
cell.detailTextLabel?.text = syncDetailText
371-
cell.imageView?.image = UIImage(named: "synced_devices")
373+
cell.imageView?.image = UIImage.templateImageNamed("synced_devices")
374+
cell.imageView?.tintColor = HistoryPanelUX.actionIconColor
375+
cell.imageView?.backgroundColor = UIColor.theme.homePanel.historyHeaderIconsBackground
376+
372377
cell.imageView?.backgroundColor = UIColor.theme.homePanel.historyHeaderIconsBackground
373378
cell.accessibilityIdentifier = "HistoryPanel.syncedDevicesCell"
374379
return cell

0 commit comments

Comments
 (0)