Comments

kalabro created an issue. See original summary.

gurvan’s picture

CSS fixes :
1) Tabs background

.cke_dialog_tabs {
    width: 100%;
    box-sizing: border-box;
    min-height: 33px;
    height: auto;
    padding: 9px 10px 0;
    margin: 0;
    left: 0;
    background: #f5f5f2;
}

2) "SELECT AN ANCHOR" floating

.cke_reset_all fieldset {
padding-top: 30px;
} 

I will publish a patch soon

gurvan’s picture

Here is the patch

gurvan’s picture

gurvan’s picture

anjalivijay’s picture

CSS fixes for anchor link.

abhisekmazumdar’s picture

Status: Active » Reviewed & tested by the community
StatusFileSize
new104.45 KB
new166.67 KB

The UI issue is fixed but I was not able to add the link. Moving this to RTBC as this issue scope was just the UI issue.

Console Error while adding the link:

rajab natshah’s picture

Status: Reviewed & tested by the community » Needs work

Thank you Kate for reporting, and thanks to Gurvan and Anjali for patching

Changing to Needs work to

  • Update the code to work with the latest version
  • Not to call active theme in all preprocess pages ( The attache of libraray could be only with the CKEditor )
soothmidas’s picture

Thank you Gurvan and Anjali for patching. I am using the Adminimal Theme and I was getting the same layering of text. I needed to include the Adminimal theme to your patch in the HOOK_preprocess_page that attaches the library. The updated patch is attached.

awolfey’s picture

Status: Needs work » Reviewed & tested by the community

I can confirm that #9 works, but it's probably better to simply check for the admin theme than a specific. Although, I suppose it is possible some site would want this to be available in a non-admin theme.

sheise’s picture

I updated the patch to not be theme specific.

sheise’s picture

Status: Reviewed & tested by the community » Needs review

Status: Needs review » Needs work
the_g_bomb’s picture

Status: Needs work » Reviewed & tested by the community

I think that failed test may be an anomaly. The test failure says "Could not read file data. Is the file empty?" for the file css/anchor_link_popup.css. But I have tested the patch and it fixes the issue as expected

adrianm6254’s picture

I applied patch #11 successfully with no issues.

rajab natshah’s picture

Status: Reviewed & tested by the community » Closed (outdated)