Combine all spellcheck settings |
||||||||||||||||
Issue descriptionFor context see c7 in crbug.com/814318 We need a better system for showing spellcheck settings because currently settings are split between 3 places and this creates confusion. E.g.: if you have server-side spellcheck enabled but local spellcheck disabled you won't get suggestions which is not intuitive. Note: There is nuance around server-side spellcheck because it will be controlled by Unity settings.
,
Jun 1 2018
,
Jun 5 2018
- When system spell check is turned off, remove the context menu item and settings rows for enhanced Google spell check. Also make sure that no data is being sent to Google (which should already be the case) - When system spell check is turned on, surface the enhanced spell check setting in three places: -- In the context menu (same way it's currently displayed) -- In the "Languages" section of settings underneath "Spell check" (see first screenshot). This row should not have a toggle; instead, it should just link to the "Sync and Google services" section of settings (need to update mock) -- In the "Sync and Google services" settings subpage, where it will be configurable (see second screenshot) Note that for Mac, system spell check is always on, so only the latter half of the above spec is relevant (i.e. the enhanced spell check settings are always visible).
,
Jun 5 2018
,
Jun 5 2018
,
Jun 5 2018
,
Jun 7 2018
Adding Joel and Johannes (discussed with them in person today)
,
Jun 7 2018
,
Jun 7 2018
Adding Namrata just in case WebUI UX expertise is needed.
,
Jun 8 2018
FYI, we've updated the spec a bit for the case where system spell check is turned OFF. We will remove the context menu item as described above, but we'll keep the rows in settings. Instead of entirely hiding them, we'll show the "Enhanced spell check" setting as disabled and OFF, and add some subtext to inform the user that they need to turn on system spell check in order to turn on "Enhanced spell check." Mocks incoming.
,
Jun 8 2018
,
Jun 8 2018
Also it turns out that we DO need to also disable/show as "OFF" enhanced Google spell check settings from both places on Mac too when the system spellcheck is disabled because if you toggle the system spellcheck on and off (system Edit menu -> Spelling&Grammar -> Check spelling while typing) it also disable enhanced spellcheck. As discussed, language team will implement: 1) Surface a pointer to the enhanced spell check setting in Language settings (mocks to come). 2) When system spell check is turned off, remove the Enhanced spellcheck context menu item on Windows, etc. (already done on Mac). DICE team will implement: 1) What Eli describes in c11 (in Sync and Google services settings, show the "Enhanced spell check" setting as disabled and OFF, and add some subtext to inform the user that they need to turn on system spell check in order to turn on "Enhanced spell check." Mocks incoming.)
,
Jun 22 2018
Issue 795710 has been merged into this issue.
,
Jul 25
,
Jul 25
,
Aug 6
,
Sep 13
,
Sep 13
,
Sep 13
,
Oct 19
Issue 868418 has been merged into this issue.
,
Nov 9
,
Nov 9
Another addition: Typing "spellcheck" in chrome://settings search bar should show all spellcheck related settings, including "Ask Google for suggestions".
,
Nov 9
Settings search only finds places where the exact string appears in the UI. It can't find synonyms. See issue 692774 for Synonyms feature request.
,
Nov 9
We've renamed the feature "Enhanced spellcheck" so theoretically if users search for spellcheck both'll show up?
,
Nov 9
Issue 891558 has been merged into this issue.
,
Nov 9
Also, dpapad@ : are you or someone on your team by any chance able to help us implement proposed changes? https://docs.google.com/presentation/d/1Ve98Q6V96ZuqTnh-WlIe0OVFWyNXT-HguYzQW_9DkbA/edit#slide=id.g4315d7706d_0_0
,
Nov 9
@yyushkina: What is the timeframe for these changes? We've already planned for Q4 and allocated resources accordingly, so I don't think we can help with the actual implementation within Q4 (unless we get some new people which is possible). Happy to help with reviews, guidance, or let's meet before Q1 2019 planning to see whether we could pick these up.
,
Nov 10
@mathp - could we get this staffed later this quarter do you think? |
||||||||||||||||
►
Sign in to add a comment |
||||||||||||||||
Comment 1 by yyushkina@chromium.org
, Jun 1 2018