chore: update documentation to refere to DruxtWrapper as DruxtTheme #358
Conversation
|
Codecov Report
@@ Coverage Diff @@
## develop #358 +/- ##
========================================
Coverage 96.70% 96.70%
========================================
Files 89 89
Lines 2826 2826
Branches 604 604
========================================
Hits 2733 2733
Misses 83 83
Partials 10 10 Continue to review full report at Codecov.
|
|
|
||
| Component options can be seen via the `component.options` data of the relevant Druxt module component. | ||
|
|
||
| If there are no matching component names, a default `DruxtWrapper` component will be used to render the default output of the module. | ||
| If there are no matching component names, a default `DruxtTheme` component will be used to render the default output of the module. |
There was a problem hiding this comment.
This is wrong, the default component is still a DruxtWrapper component.
There was a problem hiding this comment.
@BrianGilbert Still needs work.
While the concept is changing from the name Wrapper component to Theme component, the placeholder component, when no Theme component is rendered, is still called DruxtWrapper.
| @@ -63,12 +63,12 @@ The available data provided to the template scope is determined by the relevant | |||
| </DruxtEntity> | |||
| ``` | |||
|
|
|||
| By default, a component using the default template will not be wrapped by a DruxtWrapper component. It is possible to enable the DruxtWrapper system by setting the `wrapper` property to `true`: | |||
There was a problem hiding this comment.
This is also wrong, the DruxtWrapper is the placeholder component, this hasn't changed.
Types of changes
Description
Replaced reference to DruxtWrapper with DruxtTheme
Checklist:
Screenshots/Media: