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

Fix TextEditor in 2.6 Edit and Remove of Links do not work and buttons jumps #7366

Merged
merged 1 commit into from
Apr 15, 2024

Conversation

martinlagler
Copy link
Contributor

@martinlagler martinlagler commented Apr 15, 2024

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets #7355
Related issues/PRs #
License MIT

What's in this PR?

Remove blur listener for Internal- and ExternalLinkPlugin, so the edit/delete button are working again.
Removed "--ck-focus-ring: $textEditorAccentColor" variable to fix jumping issue and enhance usability.

Why?

To fix button issue and enhance usability.

@martinlagler martinlagler changed the base branch from 2.5 to 2.6 April 15, 2024 09:48
@alexander-schranz alexander-schranz changed the title Bugfix/ckeditor link Fix TextEditor in 2.6 Edit and Remove of Links do not work and buttons jumps Apr 15, 2024
@alexander-schranz alexander-schranz added the Bug Error or unexpected behavior of already existing functionality label Apr 15, 2024
@alexander-schranz alexander-schranz enabled auto-merge (squash) April 15, 2024 14:21
@alexander-schranz alexander-schranz merged commit db8103d into sulu:2.6 Apr 15, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Error or unexpected behavior of already existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants