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

FEATURE: CKE5 integration #1942

Merged
merged 21 commits into from
Jul 1, 2018
Merged

FEATURE: CKE5 integration #1942

merged 21 commits into from
Jul 1, 2018

Conversation

dimaip
Copy link
Contributor

@dimaip dimaip commented Jun 29, 2018

The default text editor is now configurable in Settings.yaml:

Neos:
  Neos:
    Ui:
      frontendConfiguration:
        defaultInlineEditor: 'ckeditor5'

TODO:

  • removeformat still missing
  • UI for creating and editing tables
  • completely rewrite link editor to improve UX

Demo:
peek 2018-07-01 11-10

@dimaip dimaip requested review from mstruebing and skurfuerst June 29, 2018 10:24
@dimaip dimaip closed this Jun 29, 2018
@dimaip dimaip reopened this Jun 29, 2018
@dimaip dimaip changed the title WIP FEATURE: CKE5 integration FEATURE: CKE5 integration Jun 29, 2018
@dimaip dimaip requested review from kdambekalns and kitsunet June 29, 2018 10:36
@kitsunet
Copy link
Member

kitsunet commented Jun 29, 2018

  • Table styles so table is always visible
  • Double click in link leads to fatal error
  • New text element has no toolbar

@kitsunet
Copy link
Member

kitsunet commented Jul 1, 2018

I could still manage to break it when I double clicked a link and then tried to apply style in another editable
screen shot 2018-07-01 at 09 21 51

Or do I have the old version still for some reason? I see the latest commit in my local branch and did make install and make build so should be latest.

@kitsunet
Copy link
Member

kitsunet commented Jul 1, 2018

Same for new element? Mmmm, might have to build again I guess...

This reverts commit fdab6ee.
@dimaip
Copy link
Contributor Author

dimaip commented Jul 1, 2018

@kitsunet works on my machine (tm)

@dimaip dimaip merged commit fbc4196 into neos:master Jul 1, 2018
@dimaip dimaip deleted the dimaip/cke5 branch July 1, 2018 07:53
@dimaip
Copy link
Contributor Author

dimaip commented Jul 1, 2018

@kitsunet I'll be away from internet for a week, but then I'll try to ping you to debug what happens on your machine, while it works perfectly fine on mine. Also please test the current state on master, maybe it has somehow got better?

@bwaidelich
Copy link
Member

Did this resolve #1889 ?

@mstruebing
Copy link
Contributor

I would say no as the CK5 integration is currently experimental and more or less hidden via a setting.

@kitsunet
Copy link
Member

kitsunet commented Jul 4, 2018

How do we deal with this problem (see thread)?
https://neos-project.slack.com/archives/C0U0KEGDQ/p1530608559000131

Can we make sure that people don't need the respective ckeditor5 theme to compile their plugins?

@mstruebing
Copy link
Contributor

mstruebing commented Jul 4, 2018

I will look into it tomorrow

@KPeschke
Copy link

KPeschke commented Jul 6, 2018

Have also tested:

  • The new link tooltip has different positions, mostly outside the node. It is currently not positioned as in your example video
  • Underline and strike through missing
  • The error reported by Christian with the double clicked link I had several times too. Unfortunately, it is difficult to reproduce the steps to cause this errors.
  • trouble with non-block elements like span. It generates < p >-tags within the span
  • missing hover and focus classes on element (blue dashed and solid outlines)
  • it seems a automated spell check is active and marks the entire text in chrome

@dimaip
Copy link
Contributor Author

dimaip commented Jul 11, 2018

Hi @KPeschke! Thanks a lot for thorough testing!

  • Link tooltip will be gone, replaced by completely custom UI.
  • Underline and strikethrough seem to work for me. Do you have the enabled in NodeTypes.yaml?
  • Will try to debug non-block elements tonight
  • Same with hover, forgot about it
  • Not sure about spell check, do you know what could be triggering it? I don't think I have it personally.

@KPeschke
Copy link

Hi @dimaip! Thanks for your reply :)

  • forgot to mention that I tested on Neos v3.3.12 with UI v1.2.2
  • Underline and strikethrough: had enabled all possible "aloha" options
  • spell check: "old" ckeditor has an attribute spellcheck="false" set

@dimaip
Copy link
Contributor Author

dimaip commented Jul 12, 2018

@KPeschke
This seems to work for me:

      ui:
        aloha:
          'format':
            'underline': true
            'strikethrough': true

Though maybe this change is still in master and not 1.2.2.

Regarding spellchecker, could be cool to be able to enable it, but we don't know the content's language...

@dimaip
Copy link
Contributor Author

dimaip commented Jul 12, 2018

Please leave your test feedback here from now on: #2002

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

Successfully merging this pull request may close these issues.

5 participants