Skip to content

Commit 8a24b89

Browse files
Merge pull request #362 from patternfly/issue-287/remove-cp-theme
Remove CP theme and references to it
2 parents c3829fa + cea0611 commit 8a24b89

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

54 files changed

+4
-1382
lines changed

docs/assets/pfe-base-sass/variables/_colors.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,7 @@ $pfe-global--disabled-state--BorderColor: $pfe-global--color--gray-mercur
160160
// Theme colors
161161
// ========================================================================
162162

163-
// I think these are misplaced. Should be in cp-theme instead.
163+
// @TODO I think these are misplaced. Should be in the theme instead
164164
// $pfe-global--primary-color: $pfe-global--color--red !default;
165165
// $pfe-global--secondary-color: $pf-color-black-600 !default;
166166
//

docs/assets/sass/_colors.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,7 @@ $pfe-global--disabled-state--BorderColor: $pfe-global--color--gray-mercur
160160
// Theme colors
161161
// ========================================================================
162162

163-
// I think these are misplaced. Should be in cp-theme instead.
163+
// @TODO I think these are misplaced. Should be in the theme instead.
164164
// $pfe-global--primary-color: $pfe-global--color--red !default;
165165
// $pfe-global--secondary-color: $pf-color-black-600 !default;
166166
//

docs/content/theme/pfe-layouts.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ tags = [ "theme" ]
1212
<link rel="stylesheet" type="text/css" href="//overpass-30e2.kxcdn.com/overpass.css">
1313
<link rel="stylesheet" type="text/css" href="../pfe-layouts.css">
1414
<link rel="stylesheet" type="text/css" href="https://cdnjs.cloudflare.com/ajax/libs/prism/1.14.0/themes/prism.min.css">
15-
<link rel="stylesheet" type="text/css" href="../../cp-themeset/cp-themeset.css">
1615

1716
<style>
1817

elements/pfe-accordion/demo/index.html

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55
<title>PFElements: pfe-accordion Demo</title>
66

77
<link rel="stylesheet" href="http://overpass-30e2.kxcdn.com/overpass.css" />
8-
<!-- <link rel="stylesheet" href="../../../themes/cp-theme/cp-theme.css" /> -->
98

109
<noscript>
1110
<link href="../../pfelement/pfelement-noscript.min.css" rel="stylesheet">

elements/pfe-autocomplete/README.md

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,15 +4,12 @@
44

55
## Install
66
```
7-
npm install @patternfly/cp-theme
87
npm install @patternfly/pfe-autocomplete
98
```
109

11-
cp-theme is a package that contains css variables setting that is used for theming patternfly elements.
1210

1311
Once installed, import it to your application:
1412
```
15-
import '@patternfly/cp-theme.umd.js';
1613
import '@patternfly/pfe-autocomplete.umd.js';
1714
```
1815

elements/pfe-autocomplete/demo/index.html

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@
1515
<script src="/@webcomponents/webcomponentsjs/webcomponents-loader.js"></script>
1616
<script src="https://cdnjs.cloudflare.com/ajax/libs/require.js/2.3.5/require.min.js"></script>
1717
<script>require([
18-
'../../../themes/cp-theme/cp-theme.umd.js',
1918
'../pfe-autocomplete.umd.js'
2019
])</script>
2120
</head>

elements/pfe-avatar/demo/index.html

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@
2121
<!-- })</script> -->
2222

2323
<script type="module" src="../pfe-avatar.js"></script>
24-
<script src="../../../themes/cp-theme/cp-theme.umd.js"></script>
2524

2625
<style type="text/css" media="screen">
2726
/* :root {

elements/pfe-avatar/demo/pfe-avatar.story.js

Lines changed: 0 additions & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

elements/pfe-card/demo/index.html

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -4,15 +4,6 @@
44
<meta charset="utf-8">
55
<title>PFElements: pfe-card Demo</title>
66

7-
<!-- needed for CP-Theme -->
8-
<link rel="stylesheet" href="http://overpass-30e2.kxcdn.com/overpass.css" />
9-
<!--
10-
if support for IE11 IS NOT needed, just include the stylesheet
11-
than contains all of the variables for your themeset
12-
-->
13-
<!-- use cp-theme -->
14-
<!-- <link rel="stylesheet" href="../../../themes/cp-theme/cp-theme.css" /> -->
15-
167
<!-- uncomment the es5-adapter if you're using the umd version -->
178
<script src="/@webcomponents/webcomponentsjs/custom-elements-es5-adapter.js"></script>
189
<script src="/@webcomponents/webcomponentsjs/webcomponents-loader.js"></script>
@@ -24,7 +15,6 @@
2415
-->
2516
<script>
2617
require([
27-
'../../../themes/cp-theme/cp-theme.umd.js',
2818
'../../pfe-cta/pfe-cta.umd.js',
2919
'../pfe-card.umd.js'
3020
]);

elements/pfe-card/demo/pfe-card.story.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,6 @@ import * as tools from "../../../.storybook/utils.js";
44

55
import PfeCard from "../pfe-card";
66

7-
// import cpTheme from "../../../themes/cp-theme/cp-theme.js";
8-
97
const stories = storiesOf("Card", module);
108

119
// Define the template to be used

0 commit comments

Comments
 (0)