Skip to content

Commit

Permalink
refactor(editor) Rectangle insertion now inserts a div.
Browse files Browse the repository at this point in the history
- `defaultRectangleElement` now creates a `div`.
- `INSERT_RECTANGLE_SHORTCUT` now doesn't include an import for `utopia-api`.
- `UtopiaApiComponents` descriptor for a `Rectangle` now creates a `div` instead.
  • Loading branch information
seanparsons committed Oct 22, 2024
1 parent fd3e0ad commit b9fdbbd
Show file tree
Hide file tree
Showing 3 changed files with 41 additions and 5 deletions.
3 changes: 2 additions & 1 deletion editor/src/components/editor/defaults.ts
Original file line number Diff line number Diff line change
Expand Up @@ -98,11 +98,12 @@ export function defaultRectangleElementStyle(): JSExpression {

export function defaultRectangleElement(uid: string): JSXElement {
return jsxElement(
jsxElementName('Rectangle', []),
jsxElementName('div', []),
uid,
jsxAttributesFromMap({
style: defaultRectangleElementStyle(),
'data-uid': jsExpressionValue(uid, emptyComments),
'data-label': jsExpressionValue('Rectangle', emptyComments),
}),
[],
)
Expand Down
4 changes: 1 addition & 3 deletions editor/src/components/editor/global-shortcuts.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -653,9 +653,7 @@ export function handleKeyDown(
EditorActions.enableInsertModeForJSXElement(
defaultRectangleElement(newUID),
newUID,
{
'utopia-api': importDetails(null, [importAlias('Rectangle')], null),
},
{},
null,
),
modifiers,
Expand Down
39 changes: 38 additions & 1 deletion editor/src/core/third-party/utopia-api-components.ts
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,43 @@ import {
jsxElementWithoutUID,
} from '../shared/element-template'

const IntrinsicComponentDescriptor = (
name: string,
insertMenuLabel: string,
supportsChildren: boolean,
styleProp: () => JSExpression,
): ComponentDescriptor => {
return {
properties: {
style: {
control: 'style-controls',
},
},
supportsChildren: supportsChildren,
preferredChildComponents: [],
variants: [
{
insertMenuLabel: insertMenuLabel,
importsToAdd: {},
elementToInsert: () =>
jsxElementWithoutUID(
name,
[
jsxAttributesEntry('style', styleProp(), emptyComments),
jsxAttributesEntry(
'data-label',
jsExpressionValue('Rectangle', emptyComments),
emptyComments,
),
],
[],
),
},
],
source: defaultComponentDescriptor(),
...ComponentDescriptorDefaults,
}
}
const BasicUtopiaComponentDescriptor = (
name: string,
supportsChildren: boolean,
Expand Down Expand Up @@ -106,7 +143,7 @@ const BasicUtopiaSceneDescriptor = (

export const UtopiaApiComponents: ComponentDescriptorsForFile = {
Ellipse: BasicUtopiaComponentDescriptor('Ellipse', false, defaultElementStyle),
Rectangle: BasicUtopiaComponentDescriptor('Rectangle', false, defaultRectangleElementStyle),
Rectangle: IntrinsicComponentDescriptor('div', 'Rectangle', false, defaultRectangleElementStyle),
View: BasicUtopiaComponentDescriptor('View', true, defaultElementStyle),
FlexRow: BasicUtopiaComponentDescriptor('FlexRow', true, defaultFlexRowOrColStyle),
FlexCol: BasicUtopiaComponentDescriptor('FlexCol', true, defaultFlexRowOrColStyle),
Expand Down

0 comments on commit b9fdbbd

Please sign in to comment.