Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Allow disabling gestures #14708

Merged
merged 1 commit into from
Sep 9, 2020
Merged

Allow disabling gestures #14708

merged 1 commit into from
Sep 9, 2020

Conversation

Mugurell
Copy link
Contributor

@Mugurell Mugurell commented Sep 4, 2020

Also prelands the Strings needed for #11862

Pull Request checklist

  • Tests: No tests. Small UI changes.
  • Screenshots: This PR includes screenshots or GIFs of the changes made.
  • Accessibility: The code in this PR follows accessibility best practices. No suggestions from the Accessibility Scanner for the "Customize" settings screen.

video showing the changes

To download an APK when reviewing a PR:

  1. click on Show All Checks,
  2. click Details next to "Taskcluster (pull_request)" after it appears and then finishes with a green checkmark,
  3. click on the "Fenix - assemble" task, then click "Run Artifacts".
  4. the APK links should be on the left side of the screen, named for each CPU architecture

@codecov-commenter
Copy link

codecov-commenter commented Sep 4, 2020

Codecov Report

Merging #14708 into master will decrease coverage by 0.00%.
The diff coverage is 27.27%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #14708      +/-   ##
============================================
- Coverage     29.84%   29.84%   -0.01%     
+ Complexity     1168     1167       -1     
============================================
  Files           445      445              
  Lines         18081    18113      +32     
  Branches       2346     2349       +3     
============================================
+ Hits           5396     5405       +9     
- Misses        12309    12332      +23     
  Partials        376      376              
Impacted Files Coverage Δ Complexity Δ
...a/org/mozilla/fenix/browser/BaseBrowserFragment.kt 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
.../java/org/mozilla/fenix/browser/BrowserFragment.kt 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
...lla/fenix/components/toolbar/BrowserToolbarView.kt 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
...rg/mozilla/fenix/settings/CustomizationFragment.kt 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
.../src/main/java/org/mozilla/fenix/utils/Settings.kt 67.26% <100.00%> (+0.67%) 28.00 <0.00> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 33fc520...9535232. Read the comment docs.

Copy link
Contributor

@mcarare mcarare left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, with just the string preference question!

Three new settings and one for which this patch just pre-lands the strings.
The behavior for the "Swipe toolbar up to open tabs" is to be added as part of
a later ticket.
@@ -282,6 +282,8 @@
<string name="preferences_theme">Theme</string>
<!-- Preference for customizing the home screen -->
<string name="preferences_home">Home</string>
<!-- Preference for gestures based actions -->
<string name="preferences_gestures">Gestures</string>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@betsymi could you confirm these strings? :)

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@ekager ekager merged commit d8d896c into mozilla-mobile:master Sep 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants