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

Mobile/iOS: Fix zoom calculation #995

Merged
merged 3 commits into from
Feb 6, 2019
Merged

Mobile/iOS: Fix zoom calculation #995

merged 3 commits into from
Feb 6, 2019

Conversation

rajivshah3
Copy link
Member

Description

Fixes zoom calculation on iOS, preventing cases where the QR scanner zooms in too much.

Type of change

  • Bug fix (a non-breaking change which fixes an issue)

How Has This Been Tested?

  • Tested manually on device

Checklist:

  • My code follows the style guidelines for this project
  • I have performed a self-review of my own code
  • For changes to mobile that include native code (including React Native modules): I have verified that both iOS and Android successfully build in both Debug and Release modes

@rajivshah3 rajivshah3 added T - Bug Type - Bug: Something isn't working P - iOS Platform - Specifically relates to iOS C - Mobile L - WIP Lifecycle - Work in Progress L - Need reviews Lifecycle - Each PR must have at least two reviewers and removed L - WIP Lifecycle - Work in Progress labels Jan 28, 2019
@rajivshah3 rajivshah3 changed the title WIP: Mobile/iOS: Fix zoom calculation Mobile/iOS: Fix zoom calculation Jan 29, 2019
Copy link
Contributor

@cvarley100 cvarley100 left a comment

Choose a reason for hiding this comment

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

Tested in debug and release 👍

@rajivshah3 rajivshah3 merged commit 9d2a3a6 into develop Feb 6, 2019
@rajivshah3 rajivshah3 deleted the fix/ios-camera branch February 6, 2019 04:09
This was referenced Feb 7, 2019
rihardsgravis pushed a commit that referenced this pull request Feb 7, 2019
* feature/realm:
  Mobile Release 0.4.1 (41) (#1029)
  New Crowdin translations [ci skip] (#1026)
  New Crowdin translations [ci skip] (#985)
  Realm data encryption (#1018)
  Bump webpack from 4.29.1 to 4.29.2 in /src/desktop (#1024)
  Bump react-dom from 16.7.0 to 16.8.0 in /src/desktop (#1022)
  Bump puppeteer from 1.12.1 to 1.12.2 in /src/desktop (#1019)
  Mobile/iOS: Fix zoom calculation (#995)
  Bump @babel/plugin-proposal-object-rest-spread in /src/desktop (#1015)
  Bump webpack from 4.29.0 to 4.29.1 in /src/desktop (#1014)
  Bump dart-sass from 1.16.1 to 1.17.0 in /src/desktop (#1016)
  Bump webpack-cli from 3.2.1 to 3.2.3 in /src/desktop (#1017)

# Conflicts:
#	src/shared/containers/components/List.js
rihardsgravis pushed a commit that referenced this pull request Feb 11, 2019
…ation

* desktop-realm-bugfixes: (107 commits)
  Add missing Windows required dependency
  Focus wallet window after initial store update
  Shared: Only rebuild Realm on Debian
  [Security] Bump extend from 3.0.1 to 3.0.2 (#1037)
  - Remove Realm instance init from Tray application - Keep Realm encryption key on keychain initialisation
  Bump react from 16.7.0 to 16.8.1 in /src/desktop (#1030)
  Bump fastlane from 2.115.0 to 2.116.0 in /src/mobile/ios (#1033)
  Bump fastlane from 2.115.0 to 2.116.0 in /src/mobile/android (#1032)
  Mobile Release 0.4.1 (41) (#1029)
  New Crowdin translations [ci skip] (#1026)
  New Crowdin translations [ci skip] (#985)
  Realm data encryption (#1018)
  Update Realm path for test environment
  - Fix Realm storage path - Add missing wallet reset triggers - Fix address component prop use - Remove failed bundle hash action
  Bump webpack from 4.29.1 to 4.29.2 in /src/desktop (#1024)
  Bump react-dom from 16.7.0 to 16.8.0 in /src/desktop (#1022)
  Bump puppeteer from 1.12.1 to 1.12.2 in /src/desktop (#1019)
  Mobile/iOS: Fix zoom calculation (#995)
  Bump @babel/plugin-proposal-object-rest-spread in /src/desktop (#1015)
  Bump webpack from 4.29.0 to 4.29.1 in /src/desktop (#1014)
  ...
rajivshah3 added a commit that referenced this pull request Mar 15, 2019
* Mobile: Bump patch-package

* Mobile/iOS: Fix zoom calculation

See react-native-camera/react-native-camera#1772

Co-authored-by: Michael Sageryd <michael@sageryd.se>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
C - Mobile L - Need reviews Lifecycle - Each PR must have at least two reviewers P - iOS Platform - Specifically relates to iOS T - Bug Type - Bug: Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants