Feat: Editing is disabled tooltip in readonly mode

[mkslanc]

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Pull Request Checklist:

• No backwards incompatible changes were made to Ace's public interface which is defined through the main typings file (ace.d.ts) and its references:
  • https://github.com/ajaxorg/ace/blob/master/ace.d.ts
  • https://github.com/ajaxorg/ace/blob/master/ace-modes.d.ts
  • https://github.com/ajaxorg/ace/blob/master/ace-extensions.d.ts

[akoreman]

We should use the nls utility to allow integrators to translate this string into whatever locale they want.

[mkslanc]

Right, totally forgot about it, added nls message and covered by tests

[codecov]

Codecov Report

Attention: Patch coverage is 97.10145% with 4 lines in your changes missing coverage. Please review.

Project coverage is 87.16%. Comparing base (2516181) to head (7fe479e).
Report is 1 commits behind head on master.

Files with missing lines	Patch %	Lines
src/editor_options_test.js	92.98%	4 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #5771      +/-   ##
==========================================
+ Coverage   87.15%   87.16%   +0.01%     
==========================================
  Files         602      603       +1     
  Lines       44157    44214      +57     
  Branches     7240     7248       +8     
==========================================
+ Hits        38484    38539      +55     
- Misses       5673     5675       +2     

Flag	Coverage Δ
unittests	87.16% <97.10%> (+0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[marinsokol5]

It's not related to this change, since we just reuse the HoverTooltip, but my VoiceOver on MacOS in Chrome doesn't read the tooltip content at all. Might work for other browser/OS/SR implementation though.

[akoreman]

@mkslanc Thanks for fixing the failing build! It seems we introduced a merge conflict with this CR, could you resolve that conflict 🙂

[mkslanc]

@mkslanc Thanks for fixing the failing build! It seems we introduced a merge conflict with this CR, could you resolve that conflict 🙂

Done :) Hope it helped