-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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 'loading' field to images in wysiwyg editor #15791
Conversation
Thank you! I wonder why URL wasn't capitalized per our Title Formatter. Also, I think one of those first two fields should be full with and we move the loading field up so the form feels more balanced. |
yep good idea, I pushed the modification :) |
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.
Thanks for your contribution 😄 lets remove api/directus-sqlite however which im assuming is an accidentally committed SQLite database.
import VSelect from '@/components/v-select/v-select.vue'; | ||
import VItem from '@/components/v-item.vue'; | ||
import VSelect1 from '@/components/v-select/v-select.vue'; | ||
import SelectDropdown from '../select-dropdown/select-dropdown.vue'; |
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.
These are not needed as v-select
is registered globally.
import VSelect from '@/components/v-select/v-select.vue'; | |
import VItem from '@/components/v-item.vue'; | |
import VSelect1 from '@/components/v-select/v-select.vue'; | |
import SelectDropdown from '../select-dropdown/select-dropdown.vue'; |
@@ -214,7 +229,7 @@ type CustomFormat = { | |||
}; | |||
|
|||
export default defineComponent({ | |||
components: { Editor }, | |||
components: { Editor, VSelect }, |
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.
components: { Editor, VSelect }, | |
components: { Editor }, |
@@ -24,6 +24,7 @@ export type SettingsStorageAssetPreset = { | |||
withoutEnlargement: boolean | null; | |||
format: 'jpeg' | 'png' | 'webp' | 'tiff' | null; | |||
transforms: any[] | null; | |||
loading: string | null; |
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.
loading: string | null; | |
loading: 'lazy' | 'eager' | null; |
Closing this for now, as the review has gone ignored |
Description
Add a
loading
field when uploading images with wysiwygFixes #15790
Type of Change
Requirements Checklist
If adding a new feature: