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.

Sign up for GitHub

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

Jump to bottom

BUG: Simply clicking into a ckeditor text field and afterwards selecting something else triggers a change #3832

Closed
grebaldi opened this issue Jul 17, 2024 · 5 comments · Fixed by #3833
Closed

BUG: Simply clicking into a ckeditor text field and afterwards selecting something else triggers a change #3832

grebaldi opened this issue Jul 17, 2024 · 5 comments · Fixed by #3833
Assignees
@grebaldi
Labels
8.3 9.0 Bug Label to mark the change as bugfix Regression A problem introduced by a recent change to the code base

Comments

@grebaldi
Copy link
Contributor

grebaldi commented Jul 17, 2024 •
edited
Loading

I fear this change introduced a regression which was not found during testing nor > is covered by e2e.
I noticed that simply clicking into a ckeditor text field and afterwards selecting > something else triggered a change.
This happens for every text element:

image

Originally posted by @mhsdesign in #3751 (comment)

To illustrate that a little more:

Peek.2024-07-17.18-27.Change.on.Deselect.mp4

Affected Versions:

Neos UI: 8.3
@grebaldi grebaldi added Bug Label to mark the change as bugfix 9.0 8.3 Regression A problem introduced by a recent change to the code base labels Jul 17, 2024
@grebaldi grebaldi mentioned this issue Jul 17, 2024
Merged
@grebaldi grebaldi self-assigned this Jul 17, 2024
@grebaldi grebaldi linked a pull request Jul 17, 2024 that will close this issue
BUGFIX: Prevent unneccessary server calls when inline editor is unchanged #3833
Merged
@crydotsnake
Copy link
Member

Can reproduce this too.

@crydotsnake
Copy link
Member

@grebaldi I still have the issue if i use the dev version of neos-ui and compiled. But not with the 8.3.9 UI release 🤔

@Sebobo
Copy link
Member

Sebobo commented Aug 5, 2024

@crydotsnake maybe some cache effect on your side

@crydotsnake
Copy link
Member

@crydotsnake maybe some cache effect on your side

I flushed the cache afterwards.

@Sebobo
Copy link
Member

Sebobo commented Aug 5, 2024

I meant build cache of the UI

@mhsdesign mhsdesign mentioned this issue Sep 5, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@grebaldi grebaldi

Labels
8.3 9.0 Bug Label to mark the change as bugfix Regression A problem introduced by a recent change to the code base
Projects
None yet
Milestone
No milestone
4 participants
@Sebobo @grebaldi @crydotsnake @mhsdesign