feat: Make translation system key based and add annotation type to gutter icon aria labels

[akoreman]

Issue #, if available: #5499

Description of changes: Changing the API we have for UI string translations from being based on the default value of the string to a more abstract key. Aim was to do this in a way that doesn't break users who currently use translations for the default value of the strings.

Changes:

• Changes main translation identifier to a more abstract key
  • Falls back to old behavior if key is missing from provided translation file
• Updates the nls step in the build accordingly
• Updates existing translation files accordingly

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

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.61%. Comparing base (507ae2f) to head (66a88a7).

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #5524   +/-   ##
=======================================
  Coverage   86.60%   86.61%           
=======================================
  Files         590      591    +1     
  Lines       42705    42732   +27     
  Branches     7096     7102    +6     
=======================================
+ Hits        36985    37012   +27     
  Misses       5720     5720           

Flag	Coverage Δ
unittests	86.61% <100.00%> (+<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.

[oykuyilmaz]

are we keeping this default string logic for backwards compatibility reasons?

[akoreman]

yeah, I kept this so that Ace users who currently have translations set-up don't get those broken by this change (well that's the goal at least 😄)