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

Spell checking language cannot be changed dynamically #2222

Closed
magreenblatt opened this issue Jul 19, 2017 · 2 comments
Closed

Spell checking language cannot be changed dynamically #2222

magreenblatt opened this issue Jul 19, 2017 · 2 comments
Labels
bug Bug report Framework Related to framework code or APIs

Comments

@magreenblatt
Copy link
Collaborator

Original report by me.


What steps will reproduce the problem?

TRY1:

  1. Only english dictionary exists at C:\Users[User]\AppData\Local\CEF\User Data\Dictionaries
  2. Run cefclient
  3. Load https://www.w3schools.com/tags/tryit.asp?filename=tryhtml_textarea
  4. Notice that spell-checking suggestions are in English.
  5. Select Tests > New Window from the test menu. In the new window, load http://tests/preferences and:
    A. Click the "Advanced" link.
    B. Change the spellcheck.dictionary value to "es".
    C. Click the "Apply Changes" button.
  6. Notice that spell-checking suggestions are no longer enabled, and es dictionary file is not downloaded.

TRY2:

  1. Run "cefclient.exe --lang=es"
  2. Load https://www.w3schools.com/tags/tryit.asp?filename=tryhtml_textarea
  3. Notice that the es dictionary file is downloaded, and that spell-checking suggestions are in Spanish.

TRY3:

  1. Run cefclient
  2. Load http://tests/preferences in the main window and:
    A. Click the "Advanced" link.
    B. Change the spellcheck.dictionary value to "es".
    C. Click the "Apply Changes" button.
  3. Select Tests > New Window from the test menu. Load https://www.w3schools.com/tags/tryit.asp?filename=tryhtml_textarea in the new window.
  4. Notice that spell-checking suggestions are in Spanish.
  5. Change the spellcheck.dictionary value to "en-US", notice that spell-checking suggestions are no longer enabled. Still not enabled if you reload or close/re-create the new window.

What is the expected output? What do you see instead?

Changing the spellcheck.dictionary value should result in download of that dictionary file if necessary and use of that language for all textareas in the same CefRequestContext. Instead, the language is "stuck" at whichever dictionary is loaded first.

What version of the product are you using? On what operating system?

Tested with 3.3071.1640.g1ebbf3c on Windows 10.

@magreenblatt
Copy link
Collaborator Author

In the SpellcheckService constructor the kSpellCheckDictionary value is converted to kSpellCheckDictionaries. Only kSpellCheckDictionaries is watched for changes. Consequently you need to modify the spellcheck.dictionaries array instead of the spellcheck.dictionary value.

@magreenblatt
Copy link
Collaborator Author

  • changed state from "new" to "wontfix"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug report Framework Related to framework code or APIs
Projects
None yet
Development

No branches or pull requests

1 participant