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

9.9.0 CKEditor config: Icon legend image files not found #4537

Closed
3 of 8 tasks
TopCat6 opened this issue Mar 2, 2021 · 6 comments · Fixed by #4546
Closed
3 of 8 tasks

9.9.0 CKEditor config: Icon legend image files not found #4537

TopCat6 opened this issue Mar 2, 2021 · 6 comments · Fixed by #4546

Comments

@TopCat6
Copy link

TopCat6 commented Mar 2, 2021

Description of bug

For CKEditor, in the Editor Config, the Icon Legend images are missing

Steps to reproduce

List the precise steps to reproduce the bug:

  1. Go to 'Site Settings, Site Behavior, More.
  2. Click on 'Open HTML Editor Manager'. Ensure DNNConnect.CKE is current Provider.
  3. See error: Left side, under Icon Legend shows that images are missing:
    image

Current behavior

Explain the current behavior.
Images for icons are missing

Expected behavior

Provide a clear and concise description of the expected behavior.
Images for icons should appear

Screenshots

If applicable, provide screenshots to help explain the bug.
image

Error information

Provide any error information (console errors, error logs, etc.) related to this bug.

Additional context

Provide any additional context that may be helpful in understanding and/or resolving the bug.
URL for first missing icon is:
/Providers/HtmlEditorProviders/DNNConnect.CKE/images/HostHasSetting.png
However, the 'images' folder does not exist in that location, and seems to be not found in any part of the tree.
The previous version, 4.5.3 in DNN 9.8.1, looked for the icon images here:
/Providers/HtmlEditorProviders/DNNConnect.CKE/js/ckeditor/4.5.3/images

Affected version

  • 10.00.00 alpha build
  • 09.09.00 latest supported release

Affected browser

  • Chrome
  • Firefox
  • Safari
  • Internet Explorer 11
  • Microsoft Edge (Classic)
  • Microsoft Edge Chromium
@james7342
Copy link

james7342 commented Mar 2, 2021

Looking at this I see a couple issues.

  1. The folders "icons" and "images” are missing from "Providers/HtmlEditorProviders/DNNConnect.CKE/js/ckeditor/4.15.1" when compared to 4.5.3
    screenshot_2089

  2. The path is incorrect as viewed from the console.
    9.8.1- https://www.example.com/Providers/HtmlEditorProviders/DNNConnect.CKE/js/ckeditor/4.5.3/images/HostHasSetting.png
    9.9.0-
    https://www.example.com/Providers/HtmlEditorProviders/DNNConnect.CKE/images/PortalHasSetting.png

I'm unable to figure out where this path is being set. So even if I copy the folders to the 4.15.1 folder the issue is still present.

@james7342
Copy link

james7342 commented Mar 3, 2021

I think I have this figured out.

  1. The folders are missing from the 4.15.1 folder structure.
  2. The pathing being incorrect is from file Providers/HtmlEditorProviders/DNNConnect.CKE/Module/EditorConfigManager.ascx

In 9.9.0 version the pathing is:

<div class="iconLegend">
            <ul>
                <li>
                    <asp:Image ID="HostHasSettingImage" runat="server" ImageUrl="~/Providers/HtmlEditorProviders/DNNConnect.CKE/images/HostHasSetting.png" />&nbsp;
                    <asp:Label runat="server" ID="HostHasSettingLabel"></asp:Label>
                </li>
                <li>
                    <asp:Image ID="HostNoSettingImage" runat="server" ImageUrl="~/Providers/HtmlEditorProviders/DNNConnect.CKE/images/HostNoSetting.png"/>&nbsp;
                    <asp:Label runat="server" ID="HostNoSettingLabel"></asp:Label>
                </li>
                <li>
                    <asp:Image ID="PortalHasSettingImage" runat="server" ImageUrl="~/Providers/HtmlEditorProviders/DNNConnect.CKE/images/PortalHasSetting.png" />&nbsp;
                    <asp:Label runat="server" ID="PortalHasSettingLabel"></asp:Label>
                </li>........

But in 9.8.1 version the pathing is:

<div class="iconLegend">
            <ul>
                <li>
                    <asp:Image ID="HostHasSettingImage" runat="server" ImageUrl="~/Providers/HtmlEditorProviders/DNNConnect.CKE/js/ckeditor/4.5.3/images/HostHasSetting.png" />&nbsp;
                    <asp:Label runat="server" ID="HostHasSettingLabel"></asp:Label>
                </li>
                <li>
                    <asp:Image ID="HostNoSettingImage" runat="server" ImageUrl="~/Providers/HtmlEditorProviders/DNNConnect.CKE/js/ckeditor/4.5.3/images/HostNoSetting.png"/>&nbsp;
                    <asp:Label runat="server" ID="HostNoSettingLabel"></asp:Label>
                </li>
                <li>
                    <asp:Image ID="PortalHasSettingImage" runat="server" ImageUrl="~/Providers/HtmlEditorProviders/DNNConnect.CKE/js/ckeditor/4.5.3/images/PortalHasSetting.png" />&nbsp;
                    <asp:Label runat="server" ID="PortalHasSettingLabel"></asp:Label>
                </li>
                <li>
                    <asp:Image ID="PortalNoSettingImage" runat="server" ImageUrl="~/Providers/HtmlEditorProviders/DNNConnect.CKE/js/ckeditor/4.5.3/images/PortalNoSetting.png"/>&nbsp;
                    <asp:Label runat="server" ID="PortalNoSettingLabel"></asp:Label>....

@james7342
Copy link

james7342 commented Mar 3, 2021

Looks to be something from this commit and PR.
#4383

Specifically this commit - "Moved DNN icons and images out of CKE folder"
af46ae9

@bdukes
Copy link
Contributor

bdukes commented Mar 3, 2021

@skamphuis it looks like the new images folder isn't getting included in the packaging. I think there needs to be a change in this build file

<Target Name="GetFiles">
<ItemGroup>
<TextFiles Include="*.txt" Exclude="license.txt;releasenotes.txt;*.Cleanup.txt" />
<Views Include="**\*.ashx;**\*.aspx;**\*.ascx" />
<CssFiles Include="css\*.css;Browser\*.css" />
<JsFiles Include="js\*.js;Browser\js\*.js;js\**\*.*" />
<ImagesFiles Include="Browser\images\**\*.*" />
<ResourceFiles Include="App_LocalResources\*.resx" />
<CKEditor Include="CKEditor\**\*.*" />
<Resources Include="@(ResourceFiles);@(TextFiles);@(CKEditor);@(Views);@(CssFiles);@(JsFiles);@(ImagesFiles);" />
</ItemGroup>
</Target>

@skamphuis
Copy link
Contributor

I think you're right. I'll have a look later on this week.

@james7342
Copy link

and icons folder..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants