Closed
Bug 905915
Opened 11 years ago
Closed 11 years ago
[Keyboard] Need to adjust the layout of keyboard FTU in landscape mode
Categories
(Firefox OS Graveyard :: Gaia::Keyboard, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: gweng, Assigned: gweng)
Details
Attachments
(3 files)
Landscape
STR:
1. With a newly Gaia installed device or set the 'keyboard.ftu.enabled' as true in settings, focus on any input element which shall invoke the keyboard
2. The FTU will pop-up with the guide of hidding the keyboard with a swiping gesture.
Expected Results:
The guide picture is at the middle of the screen and shows the picture.
Actual Results:
The picture is at the wrong position and thus invisible.
Tested On:
1. Firefox 23.0 + Gaia profile at Linux x86_64
2. 2. B2G 1.1.0-prerelease, unagi/1.1.0/beta
|
Assignee | |
Updated•11 years ago
|
Assignee: nobody → gweng
|
|
Assignee | |
Comment 1•11 years ago
|
||
Landscape mode. The picture is at the left.
|
|
Assignee | |
Comment 2•11 years ago
|
||
Another minor issue is the picture in the tip section become too small in landscape, and the screen looks like leaving too much space around the picture, so I leave a needinfo for UX.
Flags: needinfo?(nhsieh)
|
|
Assignee | |
Updated•11 years ago
|
Summary: [Keyboard] Need to adjust the layout of keyboard FTU in landscale mode → [Keyboard] Need to adjust the layout of keyboard FTU in landscape mode
|
|
Assignee | |
Comment 3•11 years ago
|
||
Found another UX issue: the original image in the tip section is portrait orientation, and it will become too high and the button will overlay on it. I can make it smaller but the space I mentioned will become larger.
|
|
Assignee | |
Comment 4•11 years ago
|
||
The image become overlayed by the button.
|
|
Assignee | |
Comment 5•11 years ago
|
||
Adjust the layout with new images and styles.
Attachment #795346 -
Flags: review?(alive)
|
||
Comment 6•11 years ago
|
||
Comment on attachment 795346 [details]
Patch
I spend one minutes to figure out "keyboard.ftu.enabled" is not "Enable keyboard in FTU app"....
Attachment #795346 -
Flags: review?(alive) → review+
|
|
Assignee | |
Comment 8•11 years ago
|
||
Oh! Since the Travis failed due to timeout problems, I just postpone the merging work and forgot to process it. Maybe [:alive] could merge it for me (I have no permission) and then I can close this bug.
Flags: needinfo?(alive)
|
|
||
Comment 9•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: needinfo?(alive)
Resolution: --- → FIXED
|
||
Comment 10•11 years ago
|
||
This should belong to keyboard component.
Greg,
Please ask me to review Gaia keyboard app related changes.
Thanks.
Component: Gaia::System → Gaia::Keyboard
|
|
||
Comment 11•11 years ago
|
||
(In reply to Rudy Lu [:rudyl] from comment #10)
> This should belong to keyboard component.
>
> Greg,
>
> Please ask me to review Gaia keyboard app related changes.
> Thanks.
/O\ Being questioned~
You need to log in
before you can comment on or make changes to this bug.
Description
•