Skip to content
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

Fixed a bug that prevented selecting folders for Browser Root and Default Upload folders in HtmlEditor #5122

Conversation

skamphuis
Copy link
Contributor

Closes #5100

Summary

This PR adds settings for

  • Browser Root Folder for images
  • Default Upload Folder for images

These settings are used when the user opens the browser for images, or uploads an image.

image

Often, it is desirable that images are kept separated from documents, and with these settings, it is no longer up to the content editor to remember to select the correct folder.

Copy link
Contributor

@donker donker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Nice feature.

@donker
Copy link
Contributor

donker commented May 13, 2022

Note: the changes in #5123 include the changes in this PR.

Copy link
Contributor

@valadas valadas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks

@valadas
Copy link
Contributor

valadas commented May 13, 2022

@donker #5123 does not have all the commits (hash) as this one. Also you put the 9.11.0 milestone, if so we need to target the release/9.11.0 branch if not, then we need to change that milestone. What way do we want to go ?

@mitchelsellers
Copy link
Contributor

@skamphuis can we get a comparison between this one #5122 and #5123, as it does not appear that this one has everything from the other, but it does have some.

Ideally, I'd like to see if we can get to a single PR for these changes

@mitchelsellers mitchelsellers changed the base branch from develop to release/9.11.0 May 24, 2022 19:16
@valadas valadas changed the title Feature/5100 cke provider file folders Fixed a bug that prevented selecting folders for Browser Root and Default Upload folders in HtmlEditor May 24, 2022
@skamphuis
Copy link
Contributor Author

@mitchelsellers I can create a single pr for them if that's easier. I based the second feature branch on the first, so I'm pretty sure the last one contains all change. But let me just create a single pr for both changes and go from there.

I will do so in the morning.

@valadas valadas merged commit 022018a into dnnsoftware:release/9.11.0 May 24, 2022
@valadas
Copy link
Contributor

valadas commented May 24, 2022

Oh, I just merged this one and will help on the other if a rebase is needed...

@skamphuis skamphuis deleted the feature/5100-CKEProvider-File-Folders branch November 23, 2022 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CKEditor missing default folder for images
7 participants