-
Notifications
You must be signed in to change notification settings - Fork 56
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add variable and expression support to serve props #895
Conversation
@@ -142,7 +142,7 @@ export default function FieldsPopover({ alignTo, show, close, onSelected, type } | |||
open={show} | |||
onClose={close} | |||
anchorEl={alignTo.current} | |||
marginThreshold={theme.spacing(1)} | |||
marginThreshold={16} // theme.spacing(1) |
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.
Fixes a MUI warning
@@ -24,7 +24,7 @@ const cfg = ({ srcDir, distDir, dev = false, serveConfig = {} }) => { | |||
eHub: [path.resolve(srcDir, 'eHub')], | |||
fixtures: [path.resolve(__dirname, './fixtures.js')], | |||
}, | |||
devtool: dev ? 'eval-cheap-module-source-map' : false, | |||
devtool: dev ? 'source-map' : false, |
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.
Source maps wheren't working correctly.
I would expect that adding |
Same here, not sure why it doesn't. WIll look into it. Have been using the config file instead. EDIT: Commands give an array as ["PP_EXPRESSION"] while we want an object { PP_EXPRESSION: true } |
Motivation
Currently the cli-serve dev environment doesn't show property panel controls for all properties. For example all q-Props are skipped. This PR adds
Where value will be a qStringExpression trying to evaluate the variable.
Requirements checklist
yarn spec
OR
When build and tests have passed: