Skip to content

Commit

Permalink
refactor: remove other warnings (#769)
Browse files Browse the repository at this point in the history
## 📜 Description

Fixed newly discovered warnings.

## 💡 Motivation and Context

In paper example I've seen even more warnings and fixed them as well 🙂
initially these warning were added in
#727

Follow up for
#766

## 📢 Changelog

<!-- High level overview of important changes -->
<!-- For example: fixed status bar manipulation; added new types
declarations; -->
<!-- If your changes don't affect one of platform/language below - then
remove this platform/language -->

### iOS

- fixed 2 warnings in `KeyboardMovementObserver`;

## 🤔 How Has This Been Tested?

Tested manually.

## 📝 Checklist

- [x] CI successfully passed
- [x] I added new mocks and corresponding unit-tests if library API was
changed
  • Loading branch information
kirillzyusko authored Jan 21, 2025
1 parent ca600a8 commit a8eb813
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions ios/observers/KeyboardMovementObserver.swift
Original file line number Diff line number Diff line change
Expand Up @@ -189,7 +189,7 @@ public class KeyboardMovementObserver: NSObject {
}

@objc func keyboardWillDisappear(_ notification: Notification) {
let (duration, keyboardFrame) = notification.keyboardMetaData()
let (duration, _) = notification.keyboardMetaData()
tag = UIResponder.current.reactViewTag
self.duration = duration

Expand Down Expand Up @@ -233,7 +233,7 @@ public class KeyboardMovementObserver: NSObject {
}

@objc func keyboardDidDisappear(_ notification: Notification) {
let (duration, keyboardFrame) = notification.keyboardMetaData()
let (duration, _) = notification.keyboardMetaData()
tag = UIResponder.current.reactViewTag

onCancelAnimation()
Expand Down

0 comments on commit a8eb813

Please sign in to comment.