Closed
Bug 1098124
Opened 10 years ago
Closed 10 years ago
Rearrange keyboard settings page scripts
Categories
(Firefox OS Graveyard :: Gaia::Keyboard, defect)
Firefox OS Graveyard
Gaia::Keyboard
Tracking
(tracking-b2g:+)
RESOLVED
FIXED
tracking-b2g | + |
People
(Reporter: timdream, Assigned: timdream)
References
Details
Attachments
(1 file)
The current keyboard settings page contains a few settings and scripts are not arranged in a way that could care each other. We will hit bugs like the page being wrongly closed by one section while the other section is downloading. I am therefore working on rearranging the script for it to follow the constructor pattern. All features touching the settings panel is probably depend on this bug.
Assignee | ||
Updated•10 years ago
|
Summary: Rearrange keyboard settings page → Rearrange keyboard settings page scripts
Assignee | ||
Comment 2•10 years ago
|
||
Comment on attachment 8522140 [details]
mozilla-b2g:master PR#26106
This patch is ready for review.
-- The UI toggles are now backed by Settings modules in js/keyboard/settings.js just like the main app.
-- These settings are represented in the UI as SettingsView.
-- Each sections has it's group view.
-- When one toggles settings we will disable and re-enable the toggle. The requests are queued in a promise chain.
-- a CloseLockManager manages each of the settings async operations so we don't close the frame in the middle of the flight. This is also important for the future downloadable and user dictionary features.
-- everything is started with KeyboardSettingsApp.
John, given the fact you will be working on user dictionary UIs, please take a look and make sure it fits the use cases. Thanks.
Attachment #8522140 -
Flags: review?(rlu)
Attachment #8522140 -
Flags: review?(jlu)
Updated•10 years ago
|
Blocks: keyboard-rtl
Comment 3•10 years ago
|
||
This also covers the RTL work for keyboard setting page.
feature-b2g: --- → 2.2+
Assignee | ||
Comment 4•10 years ago
|
||
(In reply to howie [:howie] from comment #3) > This also covers the RTL work for keyboard setting page. I can verify the Keyboard Settings page looks fine in RTL language. This bug still blocks other 2.2 works though.
Assignee | ||
Updated•10 years ago
|
No longer blocks: keyboard-rtl
Comment 5•10 years ago
|
||
Comment on attachment 8522140 [details]
mozilla-b2g:master PR#26106
I love the design of CloseLocks !
Attachment #8522140 -
Flags: review?(jlu) → review+
Comment 6•10 years ago
|
||
Comment on attachment 8522140 [details]
mozilla-b2g:master PR#26106
Looks good to me as well, only some typos.
Thanks a lot!
Attachment #8522140 -
Flags: review?(rlu) → review+
Assignee | ||
Comment 7•10 years ago
|
||
master: https://github.com/mozilla-b2g/gaia/commit/3ca836100a2c4cc2e45923f3ff0a39ede44d9507
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Comment 8•10 years ago
|
||
Can someone explain how I reach these 3 new strings, and also what "handwriting" actually does?
Assignee | ||
Comment 9•10 years ago
|
||
(In reply to Francesco Lodolo [:flod] from comment #8) > Can someone explain how I reach these 3 new strings, and also what > "handwriting" actually does? :flod, you would need to build the phone with |zh-Hans-Handwriting| layout enabled. Noted that due to bug 1098188 label will not show up unless it's not the default language. For the feature (and screenshot), see bug 1020847.
Comment 10•10 years ago
|
||
Remove the feature b2g tag to match the new v2.2 scope.
feature-b2g: 2.2+ → ---
Updated•9 years ago
|
blocking-b2g: backlog → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•