-
Notifications
You must be signed in to change notification settings - Fork 161
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
Add header to the sidebar in spaces management #8234
Conversation
6bb61bd
to
7dbd7df
Compare
983f761
to
b26d0bd
Compare
b26d0bd
to
b8ae606
Compare
|
||
export default defineComponent({ | ||
name: 'SpaceInfo', | ||
props: { | ||
spaceResource: { |
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 we call the prop just space
usually?
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.
It's kinda all over the place. A lot of these space sidebar components have a computed prop space
, so we do this to avoid name clashing.
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.
Wouldn't it make more sense to have a consistent space
var name in public component API (here: in props) and whatever other var name for internal use?
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, at least we keep the API consistent this way. But I'd do this in a follow-up as well, as it probably touches more components.
Kudos, SonarCloud Quality Gate passed! |
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.
Apart from the var name choice this is good 👍
Var name could also be solved in a followup, as it also would need changes in different components.
Description
Related Issue
Types of changes