Bug 38013 - [Patch] new Bengali Keyboard Layouts
Summary: [Patch] new Bengali Keyboard Layouts
Status: RESOLVED FIXED
Alias: None
Product: xkeyboard-config
Classification: Unclassified
Component: General (show other bugs)
Version: unspecified
Hardware: All Linux (All)
: medium enhancement
Assignee: xkb
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-06 15:49 UTC by Bryce Harrington
Modified: 2011-06-08 13:16 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
Patch to add four keyboard layouts (15.02 KB, patch)
2011-06-06 15:49 UTC, Bryce Harrington
Details | Splinter Review
Remaining evdev.xml.in changes (1.24 KB, patch)
2011-06-06 15:51 UTC, Bryce Harrington
Details | Splinter Review
Added four new Bengali Keyboard Layout (14.76 KB, patch)
2011-06-08 02:21 UTC, Promathesh Mandal
Details | Splinter Review

Description Bryce Harrington 2011-06-06 15:49:07 UTC
Created attachment 47624 [details] [review]
Patch to add four keyboard layouts

Forwarding this bug from Ubuntu reporter Promathesh Mandal:
http://bugs.launchpad.net/ubuntu/+source/xkeyboard-config/+bug/793778

Four new Bengali Keyboard Layouts are included in the attached patch

* Bengali Baishakhi
* Bengali Baishakhi Inscript
* Bengali Bornona
* Uni Gitanjali

This layouts are the most popular Bengali layouts of West Bengal (India).
Comment 1 Bryce Harrington 2011-06-06 15:51:44 UTC
Created attachment 47625 [details] [review]
Remaining evdev.xml.in changes

The original patch also had a change to evdev.xml.in which I am not seeing in the current xkeyboard-config git tree.  Not sure if that was dropped or just moved; I'll attach that chunk here just in case it's still needed.
Comment 2 Sergey V. Udaltsov 2011-06-06 17:08:03 UTC
Several things:
1. Are you sure all those variants are needed in the core set? Would it make sense to put (some of) them into base.extras.xml.in? Are they all used by many people?
2. The descriptions does not follow the existing conventions. Could you please fix?
3. Do not bother about evdev.xml.in - it is copied from base.xml.in
Comment 3 Promathesh Mandal 2011-06-08 02:21:23 UTC
Created attachment 47709 [details] [review]
Added four new Bengali Keyboard Layout

Respected Sir,
I made all changes that you are told earlier. Please review the patch.

1. All the variants are needed in the core set, because all keyboards are popular layout.
2. descriptions tag fix
3. remove evdev.xml.in
Comment 4 Sergey V. Udaltsov 2011-06-08 13:16:00 UTC
Thanks, that's great.
I took freedom to drop the variantsMapping part - there is no compatibility requirement in this case.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.