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
Typo fix and code format #2419
Typo fix and code format #2419
Conversation
docs/dev-guide/ui-api.md
Outdated
@@ -943,7 +952,7 @@ In the client addon components (browser), you have access to `$onPluginActionCal | |||
```js | |||
// Vue component | |||
export default { | |||
created () { | |||
created() { |
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.
The spaces before function parentheses are intentional.
See https://github.com/vuejs/eslint-plugin-vue-libs/blob/a3b63ec188d504e1421c9437833dd3f92fdb358a/config.js#L122
The rest of this PR is fine.
Thanks for the help!
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.
OK Sure i will update that... Thanks
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.
@sodatea updated ....
Keeping spaces between function name and parantheses
@@ -990,7 +999,7 @@ function sendMessage (data) { | |||
}) | |||
} | |||
|
|||
function messageHandler (data) { | |||
function messageHandler(data) { |
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.
Don't remove space here
@@ -1000,7 +1009,7 @@ ipc.on(messageHandler) | |||
// Don't listen anymore | |||
ipc.off(messageHandler) | |||
|
|||
function cleanup () { | |||
function cleanup() { |
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.
And here
Closing due to inactivity. |
No description provided.