delete single quote in first layer of german keyboard

VERIFIED FIXED

Status

Firefox OS
Gaia::Keyboard
VERIFIED FIXED
4 years ago
4 years ago

People

(Reporter: Anton, Assigned: gioyik, Mentored)

Tracking

unspecified

Firefox Tracking Flags

(b2g-master verified)

Details

Attachments

(2 attachments)

(Reporter)

Description

4 years ago
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
(Reporter)

Comment 1

4 years ago
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
Mentor: rlu
Flags: needinfo?(bugzilla)
(Reporter)

Comment 3

4 years ago
Yes, I am interested. You can assign me
Flags: needinfo?(bugzilla)

Comment 4

4 years ago
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!
(Assignee)

Comment 5

4 years ago
Created attachment 8546289 [details] [review]
Github PR
Attachment #8546289 - Flags: review?(rlu)
(Assignee)

Comment 6

4 years ago
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.
(Assignee)

Comment 9

4 years ago
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.
Flags: needinfo?(gioyik)
(Assignee)

Comment 11

4 years ago
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.
Keywords: checkin-needed
(Assignee)

Comment 13

4 years ago
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
Flags: needinfo?(rlu)
Keywords: checkin-needed
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
Flags: needinfo?(ktucker)
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
You need to log in before you can comment on or make changes to this bug.