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

Issue # 290 and improvements over fragment managing on settings page #301

Open
wants to merge 22 commits into
base: develop
Choose a base branch
from

Conversation

pavankumarthati
Copy link

Hi,
Issue #290 : it seems that earlier development missed to handle the orientation changes while user is seeing the "Password check dialog". As a result we are erasing the data that is stored in "noteTmp" through "save()" method in "onPause" and "onSavedInstanceState". I corrected it to handle the orientation changes.

Improvements over fragment managing on settings page : Here i saw earlier development managed the fragments by itself (i.e by storing the "backstack" and etc). I removed it using the "addbackstack" method which i feel is less error-prone going forward and easy to maintain.

Adam77Root and others added 22 commits January 6, 2016 14:01
Add setting to change first day of week in reminder calendar
Use RecyclerView to display notes list and add new staggered grid layout
# Conflicts:
#	gradle.properties
#	gradle/wrapper/gradle-wrapper.properties
#	omniNotes/src/main/java/it/feio/android/omninotes/utils/GeocodeHelper.java
#	omniNotes/src/main/res/raw/changelog.xml
#	omniNotes/src/main/res/values-de-rDE/strings.xml
Fix end position calculation for title animation in note detail view
Handled the fragment management on settings page.
@federicoiosue federicoiosue force-pushed the develop branch 2 times, most recently from cbb03b8 to 11c97fd Compare May 4, 2023 07:27
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.

None yet

4 participants