Skip to content
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

AddNoteScreen & EditNoteScreen Scroller Bug #142

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

AndchooChen
Copy link
Collaborator

Fixes #138

What was changed?

The files that were changed are the AddNoteScreen and the EditNoteScreen. Specifically the content of the return.

Why was it changed?

The issue with the current way things were was that when the editor had too much content, then it would have a difficult time showing everything that was inside of the rich editor. Thus, the way we will have to show it is by extending the editor to be large enough to show a good amount of the note.

How was it changed?

In the files stated above, the changes included increasing the overall size of the RichEditor and then also cleaning the code in areas that made that section convoluted.

Screenshots that show the changes (if applicable):

AndrewC added 3 commits April 16, 2024 12:52
… manual scrolling and also there is code to automatically ove the scroll position to where the user is typing
@AndchooChen AndchooChen added bug Something isn't working enhancement New feature or request labels Apr 17, 2024
@AndchooChen AndchooChen requested a review from yashb196 April 17, 2024 10:23
@AndchooChen AndchooChen self-assigned this Apr 17, 2024
@AndchooChen AndchooChen marked this pull request as ready for review April 22, 2024 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Note Screens Scroller Bug
1 participant