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 inline editing to record block #1282
Add inline editing to record block #1282
Conversation
transition: opacity 0.25s; | ||
} | ||
|
||
.field-label:hover .inline-actions { |
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 should be displayed when you hover over the field element, not just the label part)
@@ -224,3 +296,25 @@ export default { | |||
min-height: 1.2rem; | |||
} | |||
</style> | |||
|
|||
<style lang="scss" scoped> | |||
.field-label{ |
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.
You can do inline styles in template for this (d-flex align-items-center), the gap you dont need since you have ml-1 on the inline action anyway
} | ||
|
||
.inline-actions { | ||
width: fit-content; |
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.
I dont think you need this
cols="12" | ||
md="6" | ||
> | ||
<b-form-group |
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.
This can be in the general settings section, it belongs there.
md="6" | ||
> | ||
<b-form-group | ||
:label="$t('record.inlineEdit.enabled')" |
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 translation is missing here, should be same/similar as the one in the record list
@@ -245,6 +245,7 @@ record: | |||
listLoadFailed: Could not load record list | |||
moduleOrPageNotSet: 'RecordList block error: module or page option not set' | |||
moduleMismatch: Module incompatible, module mismatch | |||
recordLabel: Records |
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.
Since you dont need the extra section, you wont need this
todo: disable the edit btn when we're in page builder |
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.
Apply the field editor changes to all field editors that have label.
eb30bfc
to
fc6dbe8
Compare
LGTM |
8dc5664
to
3044263
Compare
LGTM |
dcc20be
to
0b9ccdb
Compare
The following changes are implemented
TODO: Summary
Changes in the user interface:
TODO: Add screenshots, recordings or remove this section
Checklist when submitting a final (!draft) PR