Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GH-1972: Triangle Bug on Dark Blue Theme and Update Copy Midnight Theme -> Dark Blue Theme #532

Merged
merged 5 commits into from Apr 26, 2020
Merged
Changes from 1 commit
Commits
File filter
Filter file types
Jump to
Jump to file
Failed to load files.

Always

Just for now

Prev
Revert "Rename midnight theme to dark blue theme"
This reverts commit f7f11f2.
  • Loading branch information
benstrumeyer committed Apr 23, 2020
commit 6c41fd511746a95077373915764e754817728c5c
@@ -1588,7 +1588,7 @@
}
},
"hub_supporter_perk_themes_description": {
"message": "Customize the Ghostery colors - try our Dark Blue theme!"
"message": "Customize the Ghostery colors - try our Midnight theme!"
},
"hub_supporter_perk_stats_description": {
"message": "View your blocking statistics and see how Ghostery is working for you"
@@ -22,7 +22,7 @@ import { RadioButtonGroup } from '../BuildingBlocks';
* @memberOf SettingsComponents
*/
const SubscriptionThemes = (props) => {
const themes = ['default', 'dark-blue-theme', 'palm-theme', 'leaf-theme'];
const themes = ['default', 'midnight-theme', 'palm-theme', 'leaf-theme'];
const themeLabels = ['subscription_default_theme', 'subscription_dark_blue_theme', 'subscription_palm_theme', 'subscription_leaf_theme'];

const getIndexClicked = () => themes.indexOf(props.theme);
@@ -1,5 +1,5 @@
/* NOTE: will need to use color variables in CSS rather than D3 code accessors
so that you can support switching between dark blue theme and regular theme */
so that you can support switching between midnight theme and regular theme */

// Function helper with color variables
@function _url-friendly-color($color) {
@@ -120,12 +120,12 @@ so that you can support switching between dark blue theme and regular theme */
background-repeat: no-repeat;
}
.point-group {
.default, .dark-blue-theme {
.default, .midnight-theme {
fill: #124559;
}
}
.line.default,
.line.dark-blue-theme {
.line.midnight-theme {
stroke: #124559;
}
}
@@ -511,7 +511,7 @@ class Metrics {
_getThemeValue() {
const { current_theme } = conf;
switch (current_theme) {
case 'dark-blue-theme':
case 'midnight-theme':
return 1;
case 'leaf-theme':
return 2;
ProTip! Use n and p to navigate between commits in a pull request.