@atlaskit/page-layout 3.4.2 → 3.4.4

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/CHANGELOG.md CHANGED
@@ -1,5 +1,18 @@
1
1
  # @atlaskit/page-layout
2
2
 
3
+ ## 3.4.4
4
+
5
+ ### Patch Changes
6
+
7
+ - [`4709c6e8fcd8a`](https://stash.atlassian.com/projects/CONFCLOUD/repos/confluence-frontend/commits/4709c6e8fcd8a) -
8
+ Bump version to remove extra html comment in style tag in React SSR
9
+
10
+ ## 3.4.3
11
+
12
+ ### Patch Changes
13
+
14
+ - Updated dependencies
15
+
3
16
  ## 3.4.2
4
17
 
5
18
  ### Patch Changes
@@ -6,14 +6,38 @@ Object.defineProperty(exports, "__esModule", {
6
6
  });
7
7
  exports.default = void 0;
8
8
  var _react = _interopRequireDefault(require("react"));
9
+ var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
9
10
  var _responsive = require("@atlaskit/primitives/responsive");
10
11
  var _default = exports.default = function _default(_ref) {
11
12
  var variableName = _ref.variableName,
12
13
  value = _ref.value,
13
14
  mobileValue = _ref.mobileValue;
14
- return (
15
- /*#__PURE__*/
16
- // eslint-disable-next-line @atlaskit/ui-styling-standard/no-global-styles -- Ignored via go/DSP-18766
17
- _react.default.createElement("style", null, ":root{--".concat(variableName, ":").concat(value, "px;}"), mobileValue && "".concat(_responsive.UNSAFE_media.below.sm, " { :root{--").concat(variableName, ":").concat(mobileValue, "px;} }"))
18
- );
15
+ if ((0, _platformFeatureFlags.fg)('platform.remove-newline-in-style-tag-in-platform-page-layout_7gts1')) {
16
+ /**
17
+ * Note don't put multiple variables in multiple lines. eg
18
+ * <style>
19
+ * {css1}
20
+ * {css2}
21
+ * </style>
22
+ *
23
+ * React will insert an empty HTML comment in between the text in SSR.
24
+ * This is not a valid tag and will break the page.
25
+ * <style>foo<!-- -->bar</style>
26
+ */
27
+ var style = ":root{--".concat(variableName, ":").concat(value, "px;}");
28
+ if (mobileValue) {
29
+ style += " ".concat(_responsive.UNSAFE_media.below.sm, " { :root{--").concat(variableName, ":").concat(mobileValue, "px;} }");
30
+ }
31
+ return (
32
+ /*#__PURE__*/
33
+ // eslint-disable-next-line @atlaskit/ui-styling-standard/no-global-styles -- Ignored via go/DSP-18766
34
+ _react.default.createElement("style", null, style)
35
+ );
36
+ } else {
37
+ return (
38
+ /*#__PURE__*/
39
+ // eslint-disable-next-line @atlaskit/ui-styling-standard/no-global-styles -- Ignored via go/DSP-18766
40
+ _react.default.createElement("style", null, ":root{--".concat(variableName, ":").concat(value, "px;}"), mobileValue && "".concat(_responsive.UNSAFE_media.below.sm, " { :root{--").concat(variableName, ":").concat(mobileValue, "px;} }"))
41
+ );
42
+ }
19
43
  };
@@ -1,10 +1,37 @@
1
1
  import React from 'react';
2
+ import { fg } from '@atlaskit/platform-feature-flags';
2
3
  import { UNSAFE_media } from '@atlaskit/primitives/responsive';
3
4
  export default (({
4
5
  variableName,
5
6
  value,
6
7
  mobileValue
7
- }) =>
8
- /*#__PURE__*/
9
- // eslint-disable-next-line @atlaskit/ui-styling-standard/no-global-styles -- Ignored via go/DSP-18766
10
- React.createElement("style", null, `:root{--${variableName}:${value}px;}`, mobileValue && `${UNSAFE_media.below.sm} { :root{--${variableName}:${mobileValue}px;} }`));
8
+ }) => {
9
+ if (fg('platform.remove-newline-in-style-tag-in-platform-page-layout_7gts1')) {
10
+ /**
11
+ * Note don't put multiple variables in multiple lines. eg
12
+ * <style>
13
+ * {css1}
14
+ * {css2}
15
+ * </style>
16
+ *
17
+ * React will insert an empty HTML comment in between the text in SSR.
18
+ * This is not a valid tag and will break the page.
19
+ * <style>foo<!-- -->bar</style>
20
+ */
21
+ let style = `:root{--${variableName}:${value}px;}`;
22
+ if (mobileValue) {
23
+ style += ` ${UNSAFE_media.below.sm} { :root{--${variableName}:${mobileValue}px;} }`;
24
+ }
25
+ return (
26
+ /*#__PURE__*/
27
+ // eslint-disable-next-line @atlaskit/ui-styling-standard/no-global-styles -- Ignored via go/DSP-18766
28
+ React.createElement("style", null, style)
29
+ );
30
+ } else {
31
+ return (
32
+ /*#__PURE__*/
33
+ // eslint-disable-next-line @atlaskit/ui-styling-standard/no-global-styles -- Ignored via go/DSP-18766
34
+ React.createElement("style", null, `:root{--${variableName}:${value}px;}`, mobileValue && `${UNSAFE_media.below.sm} { :root{--${variableName}:${mobileValue}px;} }`)
35
+ );
36
+ }
37
+ });
@@ -1,12 +1,36 @@
1
1
  import React from 'react';
2
+ import { fg } from '@atlaskit/platform-feature-flags';
2
3
  import { UNSAFE_media } from '@atlaskit/primitives/responsive';
3
4
  export default (function (_ref) {
4
5
  var variableName = _ref.variableName,
5
6
  value = _ref.value,
6
7
  mobileValue = _ref.mobileValue;
7
- return (
8
- /*#__PURE__*/
9
- // eslint-disable-next-line @atlaskit/ui-styling-standard/no-global-styles -- Ignored via go/DSP-18766
10
- React.createElement("style", null, ":root{--".concat(variableName, ":").concat(value, "px;}"), mobileValue && "".concat(UNSAFE_media.below.sm, " { :root{--").concat(variableName, ":").concat(mobileValue, "px;} }"))
11
- );
8
+ if (fg('platform.remove-newline-in-style-tag-in-platform-page-layout_7gts1')) {
9
+ /**
10
+ * Note don't put multiple variables in multiple lines. eg
11
+ * <style>
12
+ * {css1}
13
+ * {css2}
14
+ * </style>
15
+ *
16
+ * React will insert an empty HTML comment in between the text in SSR.
17
+ * This is not a valid tag and will break the page.
18
+ * <style>foo<!-- -->bar</style>
19
+ */
20
+ var style = ":root{--".concat(variableName, ":").concat(value, "px;}");
21
+ if (mobileValue) {
22
+ style += " ".concat(UNSAFE_media.below.sm, " { :root{--").concat(variableName, ":").concat(mobileValue, "px;} }");
23
+ }
24
+ return (
25
+ /*#__PURE__*/
26
+ // eslint-disable-next-line @atlaskit/ui-styling-standard/no-global-styles -- Ignored via go/DSP-18766
27
+ React.createElement("style", null, style)
28
+ );
29
+ } else {
30
+ return (
31
+ /*#__PURE__*/
32
+ // eslint-disable-next-line @atlaskit/ui-styling-standard/no-global-styles -- Ignored via go/DSP-18766
33
+ React.createElement("style", null, ":root{--".concat(variableName, ":").concat(value, "px;}"), mobileValue && "".concat(UNSAFE_media.below.sm, " { :root{--").concat(variableName, ":").concat(mobileValue, "px;} }"))
34
+ );
35
+ }
12
36
  });
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@atlaskit/page-layout",
3
- "version": "3.4.2",
3
+ "version": "3.4.4",
4
4
  "description": "A collection of components which let you compose an application's page layout.",
5
5
  "publishConfig": {
6
6
  "registry": "https://registry.npmjs.org/"
@@ -41,10 +41,10 @@
41
41
  "dependencies": {
42
42
  "@atlaskit/ds-lib": "^2.3.0",
43
43
  "@atlaskit/icon": "^22.5.0",
44
- "@atlaskit/link": "^0.5.0",
44
+ "@atlaskit/link": "^1.0.0",
45
45
  "@atlaskit/motion": "^1.7.0",
46
- "@atlaskit/platform-feature-flags": "^0.2.0",
47
- "@atlaskit/primitives": "^9.0.0",
46
+ "@atlaskit/platform-feature-flags": "^0.3.0",
47
+ "@atlaskit/primitives": "^10.0.0",
48
48
  "@atlaskit/theme": "^12.11.0",
49
49
  "@atlaskit/tokens": "^1.53.0",
50
50
  "@babel/runtime": "^7.0.0",
@@ -59,14 +59,14 @@
59
59
  "devDependencies": {
60
60
  "@af/accessibility-testing": "*",
61
61
  "@af/integration-testing": "*",
62
- "@atlaskit/atlassian-navigation": "^4.1.0",
62
+ "@atlaskit/atlassian-navigation": "^4.2.0",
63
63
  "@atlaskit/atlassian-notifications": "^0.4.0",
64
64
  "@atlaskit/drawer": "^7.12.0",
65
65
  "@atlaskit/logo": "^14.1.0",
66
- "@atlaskit/menu": "^2.5.0",
66
+ "@atlaskit/menu": "^2.6.0",
67
67
  "@atlaskit/notification-indicator": "^9.2.0",
68
68
  "@atlaskit/notification-log-client": "^6.1.0",
69
- "@atlaskit/popup": "^1.19.0",
69
+ "@atlaskit/popup": "^1.20.0",
70
70
  "@atlaskit/side-navigation": "^3.3.0",
71
71
  "@atlaskit/ssr": "*",
72
72
  "@atlaskit/toggle": "^13.2.0",
@@ -119,6 +119,9 @@
119
119
  },
120
120
  "platform.atlassian.griffin-navigation-redesign": {
121
121
  "type": "boolean"
122
+ },
123
+ "platform.remove-newline-in-style-tag-in-platform-page-layout_7gts1": {
124
+ "type": "boolean"
122
125
  }
123
126
  }
124
127
  }