[go: nahoru, domu]

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

Fix for issue #170 "Edit Page always asks to save changes". #278

Closed
wants to merge 1 commit into from
Closed

Conversation

ShiftLeftLogical
Copy link

The save dialog will no longer pop when a page/post content is not modified and the back button is pressed.

…dialog

will no longer pop when a page/post content is not modified and the
back button is pressed.
@ShiftLeftLogical
Copy link
Author

Please do not use this pull request as it is not as good of a solution as my more recent fix for issue #170 because this fix could fail to pop the dialog when content had changed (with an extremely small probability of that ever happening.) The only advantage this solution had was that it did not need to store duplicates of the initial post content but I do not feel this is something that I needed to optimize working memory for, in this case.

maxme added a commit that referenced this pull request Feb 16, 2016
4832795 Merge pull request #280 from wordpress-mobile/issue/remove-analytics-dependency
dd68ccd Merge pull request #283 from wordpress-mobile/issue/282-localize-js-strings
bec42a8 moved the brackets back into the JS code
2cd15b7 fix #282: localize JS strings by using js-java interface nativeState
a64d3a7 Merge pull request #270 from wordpress-mobile/issue/242-image-container-lower-api
5339a22 Fixed some EditorFragment lint issues
ab084be Used a String constant for naming the native state JS interface
0789069 Merge branch 'develop' into issue/242-image-container-lower-api
430d3a4 Merge pull request #278 from wordpress-mobile/issue/183-accessibility
e4b9e76 s/Strike/Strikethrough/
ec9c6e9 Image thumbnail content description
cda2f22 Make editor title i18nizable
9d92717 Merge branch 'develop' into issue/242-image-container-lower-api
67dec0b Change page title for accessibility
3167e9a add contentDescription to format bar buttons in the visual and legacy editors
29cea46 Fixed issue where compatibility upload overlays would sometimes consume a tap without triggering a tap callback for the associated image
7b26aa7 Fixed a display issue on API<19 where a dark blank space would appear below an uploading or failed image
b95d94a Added a dark semi-transparent overlay for API<19 for uploading and failed images
9e1863f Revert to non-compatibility image container styling once image has begun uploading
34523c0 Hide the compatibility image upload overlay if an image upload fails and re-display it on resume
abf7260 Use block instead of inline-block for the image container on pre=KitKat Android
86a985c On pre-KitKat Android, show an 'Uploading...' overlay instead of a progress bar for uploading images
047082a Updated ZSSEditor to get the current Android API level on init

git-subtree-dir: libs/editor
git-subtree-split: 48327953c76c593daefe93a16e41323631de5b73
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

1 participant