MathJax-src icon indicating copy to clipboard operation
MathJax-src copied to clipboard

Fixes issues with firefox and safari.

Open zorkow opened this issue 1 year ago • 4 comments

These are the workarounds for the issues regarding voice events on Firefox. @dpvc can you have a look if this works for Safari as well?

zorkow avatar Feb 15 '24 04:02 zorkow

Here are the list of continuing issues from our chat today:

  • Hover highlighting leads to an error.
  • While explorer is active, all key events are blocked (including reload, etc.)
  • Can't start explorer if both speech and braille are unchecked. I believe that this is due to the fact that there are no role attributes in that case, because those seem to be added during the speech/braille creationSpeech() function in semantic-enrich, which isn't being called.
  • After a change in language, you must do a move to get new language (but you know this one already).

dpvc avatar Feb 19 '24 21:02 dpvc

I figured out the third issue above, and have fixed it in my menu update branch (not yet pushed).

dpvc avatar Feb 21 '24 21:02 dpvc

Here are the list of continuing issues from our chat today:

  • Hover highlighting leads to an error.

This was a CHTML error in SRE, which is now being caught (latest develop branch). But note that hover only works when the explorer is active. Otherwise we do not have a highlighter as those are only created when we have an ExplorerItem. We can probably change that in the future, but I am not sure if it is worth it.

  • While explorer is active, all key events are blocked (including reload, etc.)

That is now no longer the case.

  • Can't start explorer if both speech and braille are unchecked. I believe that this is due to the fact that there are no role attributes in that case, because those seem to be added during the speech/braille creationSpeech() function in semantic-enrich, which isn't being called.

That was actually an issue related to how explorers were attached. Previously, speech, braille and keyMagnifier would attach their own explorer, now only speech attaches the SpeechExplorer which handles all three duties. So we need to see if at least one of the three options in options.a11y is set, to attach and thus enable the SpeechExplorer.

  • After a change in language, you must do a move to get new language (but you know this one already).

We now have a similar issue with switching speech/braille before retypesettting. This is best fixed once the new Menu structure is integrated.

zorkow avatar Mar 06 '24 17:03 zorkow

That was actually an issue related to how explorers were attached.

I fixed it in the sre-menus branch, as I mentioned above, so you don't need to worry about that one.

dpvc avatar Mar 06 '24 17:03 dpvc

After a change in language, you must do a move to get new language (but you know this one already).

This is now fixed. However, we still have an issue when changing rule sets/preference settings. I will tackle this in a new PR to simplify review.

zorkow avatar Mar 28 '24 13:03 zorkow

@dpvc as discussed today, this should be ready to merge.

zorkow avatar Mar 28 '24 23:03 zorkow