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

Restore standard status for config.disableStyleCopyPaste #9344

Open
asturur opened this issue Sep 15, 2023 · 5 comments
Open

Restore standard status for config.disableStyleCopyPaste #9344

asturur opened this issue Sep 15, 2023 · 5 comments
Assignees
Labels

Comments

@asturur
Copy link
Member

asturur commented Sep 15, 2023

I was using again disableStyleCopyPaste and i found it deprecated and i was surprised.
It happened here: #8194
I checked back the PR but i didn't find a reason for it.

We should remove the deprecation notice.

@asturur asturur self-assigned this Sep 15, 2023
@asturur asturur added the v6 label Sep 15, 2023
@ShaMan123
Copy link
Contributor

It is because it was supposed to be removed in #8942

@asturur
Copy link
Member Author

asturur commented Sep 15, 2023

There are situations in which you don't want to deal with style at all, i don't remember that PR offering an alternative for disabling it.

@ShaMan123
Copy link
Contributor

It does, easily.
you reset the event data in a simple listener, I belive I wrote that done and/or added a test and if not then I should

@asturur
Copy link
Member Author

asturur commented Sep 15, 2023

no please don't. We are not doing radical text code changes now, and when we do we can totally keep the boolean option.
When we are done with 6.0 we can find a way to improve the copy/paste events so that we can intercept or change the data but this option shouldn't disappear because of that.

@ShaMan123
Copy link
Contributor

It is not now, it is an old PR that introduces the ability to copy paste from outside og the tab, which is more than important for apps I use

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

No branches or pull requests

2 participants