-
-
Notifications
You must be signed in to change notification settings - Fork 100
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve Device Name in Firefox/Mozilla Sync #1071
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A bit busy to review it properly. Just a couple of nits I quickly spotted
5e2d306
to
b088379
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works pretty well. I just have a couple of nits to be addressed before landing. Thanks!
app/src/common/shared/com/igalia/wolvic/ui/widgets/settings/FxAAccountOptionsView.java
Show resolved
Hide resolved
app/src/common/shared/com/igalia/wolvic/ui/widgets/settings/FxAAccountOptionsView.java
Outdated
Show resolved
Hide resolved
app/src/common/shared/com/igalia/wolvic/ui/widgets/settings/FxAAccountOptionsView.java
Outdated
Show resolved
Hide resolved
fac6f26
to
041adc0
Compare
Use the pre-defined device name according to device type. If the device type is unknown (i.e. no-api), use Build.MANUFACTURER + " " + Build.MODEL as fall back. Signed-off-by: Songlin Jiang <sjiang@igalia.com>
Resolve #1069 Signed-off-by: Songlin Jiang <sjiang@igalia.com>
So that we don't need to scroll for device name in accounts settings. Signed-off-by: Songlin Jiang <sjiang@igalia.com>
32e455c
to
5d631ca
Compare
I just fixed the rebase error @svillar |
Use the pre-defined device name according to device type. If the device type is unknown (i.e. no-api), use Build.MANUFACTURER + " " + Build.MODEL as fall back.
So that we don't need to scroll for newly added haptic feedback in controllers settings in Enable haptic feedback for controllers #1068 and device name in accounts settings.
Resolves #1069