roadmap icon indicating copy to clipboard operation
roadmap copied to clipboard

Ensure spell check is enabled for smaller data entry boxes

Open magdalenadrafiova opened this issue 6 years ago • 14 comments

From our accessibility testing the following was mentioned: The smaller text boxes e.g. what project are you planning did not have spellcheck enabled but the larger free text boxes e.g. Project Abstract did.

magdalenadrafiova avatar Jul 09 '19 15:07 magdalenadrafiova

I'm not 100% sure how to test this one. On one plan I have I can see 'NERC' underlined in red in a project details abstract. This may be coming from browser rather than an accessibility feature in tool.

Any advice @johnpinto1

sjDCC avatar Sep 28 '19 05:09 sjDCC

I am sorry I do not know how to test this one - I am adding this to our meeting on Tuesday morning so Sam/Ray can show us

magdalenadrafiova avatar Nov 22 '19 12:11 magdalenadrafiova

@sjDCC @magdalenadrafiova spellcheck: "true" is a HTML 5 attribute that has been added to all text field tags. So the browser if it supports HTML 5 will highlight spelling errors for your locale.

johnpinto1 avatar Nov 22 '19 12:11 johnpinto1

Now when I go into create a new project and I start filling the details there is a spell check enabled for the front page - for the title of the new project. Seems fine closing this. @sjDCC @xsrust

magdalenadrafiova avatar Nov 26 '19 12:11 magdalenadrafiova

Reopening b/c we're not seeing the feature in tinymce boxes. Check plugins to enable spellcheck plugin. John on enabled browser spellcheck

mariapraetzellis avatar Jan 07 '20 17:01 mariapraetzellis

Having a look at the types of spellchecking that TinyMCE supports, I think I'd still rather go with the browser-based package rather than us running a spellchecking service. See: https://www.tiny.cloud/docs/general-configuration-guide/spell-checking/ for more details on the options given to us by TinyMCE.

xsrust avatar Jan 31 '20 12:01 xsrust

I'm not clear if this was enabled @xsrust? I don't see the spell check feature in data entry boxes.

I have heard from several users that they would like to see a spell check feature, so unless it's a real drag on the application I think it is worth enabling. My Chrome browser was not picking up spelling errors when filling out a test DMP in the app.

screenshot-dmponline-dev dcc ac uk-2020 02 03-15_09_20

mariapraetzellis avatar Feb 03 '20 23:02 mariapraetzellis

that has not been deployed just yet

magdalenadrafiova avatar Feb 04 '20 16:02 magdalenadrafiova

seems to be working @mariapraetzellis you want to try too? screenshot-dmponline-dev dcc ac uk-2020 02 04-17_04_21

magdalenadrafiova avatar Feb 04 '20 17:02 magdalenadrafiova

Looks good for me too. Closing out

sjDCC avatar Feb 10 '20 16:02 sjDCC

Looks good to me as well.

mariapraetzellis avatar Feb 10 '20 16:02 mariapraetzellis

Sorry to have to re-open this but the spell checker still doesn't work for all data entry boxes. For example, notification title box (it works for the body box).

dsisu avatar Jun 04 '21 16:06 dsisu

Will look at this now.

johnpinto1 avatar Sep 30 '21 09:09 johnpinto1

@dsisu this works for me now, are you okay to confirm and close out or update with details of where you still encounter issues please?

pherterich avatar Jun 23 '22 09:06 pherterich