-
Notifications
You must be signed in to change notification settings - Fork 27
Fix Various Issues With The fileorganizer Dialog #18
Fix Various Issues With The fileorganizer Dialog #18
Conversation
This replaces the existing behavior of displaying "Error: null"
dlg.querySelector('#selectMedias').setAttribute('label', 'Movie'); | ||
dlg.querySelector('[for="selectMedias"]').innerHTML = 'Movie'; | ||
mediaSelector.setAttribute('label', 'Movie'); | ||
if (mediaSelector && mediaSelector.setLabel) mediaSelector.setLabel('Movie'); |
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.
When the dialog first opens, the <label>
element for the selector has not yet been created so we check for that here. The label text is still initialized correctly by the emby-select
component when it loads using the label
attribute value. See: https://github.com/jellyfin/jellyfin-web/blob/master/src/components/emby-select/emby-select.js#L151
} | ||
|
||
// Show dialog | ||
dialogHelper.open(dlg); |
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.
Open the dialog only after all other initialization has completed so that it is ready to be viewed
Alright - Now that the plug-in has been updated with the other reps, I'll review this one, and if it's good, we'll cut a release. |
This PR addresses several issues with the fileorganizer dialog:
NOTE: The version number for the plugin has not been updated here. I'm not sure how releases are performed and how changes should be coordinated with the existing PR #17