This adds missing Alt Gr mappings for the keys "q", "w", "e" an "c" to conform with ABNT2 standard [1]
PR: 256416
Submitted by: Neebz <vpguyrhpjltta@logicstreak.com>
MFC after: 2 weeks
Paths
| Differential D33801 Authored by alfredo on Jan 9 2022, 5:10 AM.
Details Summary This adds missing Alt Gr mappings for the keys "q", "w", "e" an "c" to conform with ABNT2 standard [1] PR: 256416 Test Plan test output using an ABNT/ABNT2 keyboard (Brazillian Portuguese) on the following USB-wireless keyboards :
-Dell WK636t And the following Laptop:
Diff Detail
Event TimelineHerald added subscribers: Contributor Reviews (src), imp. · View Herald TranscriptJan 9 2022, 5:10 AM2022-01-09 05:10:44 (UTC+0) Harbormaster completed remote builds in B43795: Diff 101160.Jan 9 2022, 5:10 AM2022-01-09 05:10:44 (UTC+0) alfredo edited the test plan for this revision. (Show Details)Feb 22 2022, 7:07 PM2022-02-22 19:07:03 (UTC+0) Comment Actions As I mentioned in the PR I am not familiar with Brazilian keyboards, but the patch looks fine. This revision is now accepted and ready to land.Feb 22 2022, 9:17 PM2022-02-22 21:17:59 (UTC+0) Closed by commit rG07625bc4b30d: keymap: Add extra Alt Gr mapping for Brazillian Portuguese ABNT2 keyboards (authored by alfredo). · Explain WhyFeb 22 2022, 9:43 PM2022-02-22 21:43:38 (UTC+0) This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 103124 share/vt/keymaps/br.kbd
share/vt/keymaps/br.noacc.kbd
|
There might be a micro optimization here to do the V_broadcast_zero test LAST, that way the additional test in the fast path only occurs after the tests we have been doing all along.