-
Notifications
You must be signed in to change notification settings - Fork 13
[terra-dev-site] Terra dev site v7 #255
Changes from 37 commits
2b32c26
4c2fc70
efe7bba
bc25baf
0458d06
dc6897c
7510f1e
2f9bb36
7667b8f
3293a2c
de0b1f9
2889f4c
70f0887
7fd50d4
592e7b5
a268df9
5c8b2e8
572d402
dc8d5cf
714dd22
72e525f
01ad973
1cb56ef
8479db3
233988c
78a555a
8976e98
3498c27
ae97318
208dc7e
a475993
25036c5
7080ca7
161846f
808dd45
5aaa5e2
6c507c9
dcd6176
7ddf504
bf369a7
5d3a275
905159f
78876e6
d32f3ca
6fcb78c
4cb711f
a0341a9
c04bd4b
256fae2
efe955e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2,6 +2,9 @@ | |
|
||
## Unreleased | ||
|
||
* Added | ||
* terra-dev-site v7 docs | ||
|
||
## 1.1.0 - (August 11, 2021) | ||
|
||
* Changed | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
# Additional Search Directory |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
import React from 'react'; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Do files in this folder need to have There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yeah it does, this is just for testing purposes, we could rename things but it doesn't really matter since it's just for testing. |
||
|
||
export default () => ( | ||
<h1 className="terra-dev-site-extended"> | ||
extended | ||
</h1> | ||
); |
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
:global { | ||
body, | ||
h1 { | ||
height: 100%; | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,25 @@ | ||
import { Notice } from '@cerner/terra-docs'; | ||
|
||
# FAQ | ||
|
||
## My components are visible in the raw test route but cut off when displayed in the dev site frame | ||
|
||
When components are displayed in the raw route there is an unbroken chain of height 100%s set on the components such that test components with 100% height will fill the available area. With pages, for aesthetic reasons we do not provide a 100% height on all wrapping components. To ensure components display correctly in both raw routes and pages either use block layout or define a fixed hight above any components expecting 100% height. | ||
|
||
## I get the webpack error: Invalid configuration object. Webpack has been initialised using a configuration object that does not match the API schema. | ||
|
||
You can get this error when using webpack 4. Part of the TerraDevSite webpack plugin we inject an entry to build the site. Webpack requires an entry to function but when the check is performed changed between webpack 4 and 5. In webpack 4 an entry is checked prior to running the plugin where as in webpack 5 the check is performed after. | ||
|
||
To solve this issue you have two options. Upgrade to webpack 5 or add a dummy entrypoint that TerraDevSite will remove before bundling. Upgrading to webpack 5 is recommended. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Could we add a dummy entry point for teams? I'm not sure I like that idea, just throwing it out there. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I thought about providing the entrypoint, but figured giving them the code snipit was good enough. My reasoning is that it's only needed for webpack 4 and my preferred solution would be for them to upgrade to webpack 5. I don't have a really strong opinion and would be open to offering the entrypoint if someone feels strongly about it. |
||
|
||
<Notice variant="important" ariaLevel="3"> | ||
|
||
If you chose to add the dummy entry point it will have to resolve to a real file. It doesn't matter what file since TerraDevSite will remove it, but webpack 4 does check that the entrypoint file is real. | ||
|
||
</Notice> | ||
|
||
```js | ||
entry: { | ||
terraDevSiteDummy: './dummy.js', | ||
}, | ||
``` |
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm rusty, will this bump with the release?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah it'll get updated with the prepare for release script