Skip to content

Improve keyboard on notes screens ios android#262

Open
InfinityBowman wants to merge 13 commits intomainfrom
Improve-keyboard-ios-android
Open

Improve keyboard on notes screens ios android#262
InfinityBowman wants to merge 13 commits intomainfrom
Improve-keyboard-ios-android

Conversation

@InfinityBowman
Copy link
Copy Markdown
Collaborator

@InfinityBowman InfinityBowman commented Sep 20, 2025

Lots of weird scroll behavior that I think I sorted out. The toolbar should be correctly placed above the keyboard with a close button on both IOS and Android on both the AddNoteScreen and EditNoteScreen. I also think that there could be something done in the future to combine parts of the AddNoteScreen and EditNoteScreen since they share a lot of common code. Try it out to make sure I didn't break anything.

Also added prettierrc and fixed a tsconfig error

dispatch(toogleAddNoteState());
}
}, 300); // allow WebView to flush
}, 300); // allow WebView to flush (note: react native doesnt use a webview - jacob)
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

react native does not however in some features ( video/image ) in expo it does use react-native-webview

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Oh, I see that tentap uses a webview internally for some things

Copy link
Copy Markdown
Collaborator

@yashcoded yashcoded left a comment

Choose a reason for hiding this comment

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

WhatsApp Image 2025-09-21 at 14 21 41_39426b8c
check this error out shows up whenever you run yarn start

@InfinityBowman
Copy link
Copy Markdown
Collaborator Author

WhatsApp Image 2025-09-21 at 14 21 41_39426b8c check this error out shows up whenever you run yarn start

Is this after you did any updating of expo? I had this same thing but I changed branches and did yarn install and then it works fine again

@yashcoded
Copy link
Copy Markdown
Collaborator

WhatsApp Image 2025-09-21 at 14 21 41_39426b8c check this error out shows up whenever you run yarn start

Is this after you did any updating of expo? I had this same thing but I changed branches and did yarn install and then it works fine again

after the package update yes try it out on your device i tried it on 2 devices still the error persists

@InfinityBowman
Copy link
Copy Markdown
Collaborator Author

WhatsApp Image 2025-09-21 at 14 21 41_39426b8c check this error out shows up whenever you run yarn start

Is this after you did any updating of expo? I had this same thing but I changed branches and did yarn install and then it works fine again

after the package update yes try it out on your device i tried it on 2 devices still the error persists

Yeah updating expo to 54 must have some breaking change for firebase.

@InfinityBowman
Copy link
Copy Markdown
Collaborator Author

InfinityBowman commented Sep 21, 2025

Well something with the testing library is broken now

@mhashir03 mhashir03 self-assigned this Oct 27, 2025
@mhashir03
Copy link
Copy Markdown
Collaborator

using this pr to work on issue 263

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants