Created attachment 8525543 [details] german-keyboard.png User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:33.0) Gecko/20100101 Firefox/33.0 Build ID: 20141013195847 Steps to reproduce: 1. open german keyboard Actual results: first layer of keyboard appers with letters, single quote and dot. the single quote makes the second row of the first layer broader and malformed. therefore it's more difficult to type. In addition, there is already a single quote on the second layer. Expected results: first layer of keyboard should apper without single quote
in case this is beginner friendly bug a would like to fix it, too
According to this comment, it should be ok to remove it from the first (default) page of German layout, bug 826038 comment 9. And the code lives in, https://github.com/mozilla-b2g/gaia/blob/ba0337ffddb62ef3cf388b904e25966d30c89609/apps/keyboard/js/layouts/de.js#L28 Anton, are you interested in fixing this issue? If you need more information for fixing this, please help set needinfo to me. If this is the first time for you to work on Gaia issue, you could start from here, https://github.com/mozilla-b2g/gaia/blob/master/CONTRIBUTING.md
Yes, I am interested. You can assign me
Hi, Anton - we can assign this to you as soon as you push up a first try at a patch. Let us know how we can help you do that!
It's ok the PR submitted or more changes are needed?
Comment on attachment 8546289 [details] [review] Github PR Looks good. Thanks.
Attachment #8546289 - Flags: review?(rlu) → review+
Assignee: nobody → gioyik
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
There are some extra space in the layout definition file, please help fix them in your patch if you could help. Please just leave them as is if you don't know what to do and set "checkin-needed" in the keyword to ask for checking in your patch. Thanks.
Rudy, I deleted the extra space. Could you check it and tell me if is ok? Thanks
There is another extra space at Line 23. BTW, could you squash the commits? Thanks.
Changes on Line 23 are done and the PR has squashed commits. Is everything ok? Thanks
Flags: needinfo?(gioyik) → needinfo?(rlu)
Yes, thank you very much for your work! Now our source tree is closed, so that I cannot merge your patch. Will set checkin-needed first and keep the ni as is.
Thank you. Let me know if there's something where I can help.
Merged to Gaia master, https://github.com/mozilla-b2g/gaia/commit/6a9e87729aff9534f00afba7367eb4aded4e51e2
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
status-b2g-master: --- → fixed
Resolution: --- → FIXED
This issue is verified fixed on Flame Master. Result: The single quote does not exist on the first layer of the German keyboard. Device: Flame Master (319mb, full flash) Build ID: 20150129010239 Gaia: 9d2378a9ef092ab1fc15c3a9f7fc4171aab59d57 Gecko: 6bfc0e1c4b29 Gonk: e7c90613521145db090dd24147afd5ceb5703190 Version: 38.0a1 (3.0) Firmware Version: v18D-1 User Agent: Mozilla/5.0 (Mobile; rv:38.0) Gecko/38.0 Firefox/38.0
Status: RESOLVED → VERIFIED
QA Whiteboard: [QAnalyst-Triage?]
status-b2g-master: fixed → verified
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
You need to log in before you can comment on or make changes to this bug.