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 6 times, most recently from f42d265 to cfbdbc4 Compare August 4, 2020 15:52
@federicoiosue federicoiosue force-pushed the develop branch 2 times, most recently from 73fc7a0 to c87d5ee Compare August 5, 2020 09:58
@federicoiosue federicoiosue force-pushed the develop branch 4 times, most recently from cfb925a to db0286b Compare August 14, 2020 22:04
@federicoiosue federicoiosue force-pushed the develop branch 2 times, most recently from 0f1f367 to c8987f1 Compare October 2, 2020 10:17
@federicoiosue federicoiosue force-pushed the develop branch 2 times, most recently from d78a98e to 0168039 Compare December 23, 2020 09:31
@federicoiosue federicoiosue force-pushed the develop branch 5 times, most recently from 911df26 to a6655ce Compare October 27, 2022 22:29
@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.

4 participants