Closed
Bug 402275
Opened 17 years ago
Closed 16 years ago
Don't nag about leaving an unsaved page if no changes have been made
Categories
(support.mozilla.org :: Knowledge Base Software, task)
support.mozilla.org
Knowledge Base Software
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: jason.barnabe, Assigned: jason.barnabe)
Details
(Whiteboard: tiki_test)
Attachments
(1 file)
1.77 KB,
patch
|
djst
:
review-
|
Details | Diff | Splinter Review |
If you go to edit a page and then navigate somewhere else without saving, the site will ask you whether you really want to do that. It shouldn't ask unless you've actually made some changes.
Assignee | ||
Comment 1•17 years ago
|
||
With this patch, the user will not be prompted for confirmation if they're on the plain edit screen (not the preview screen) and the text of the main textarea has not changed. This patch assumes jQuery is available at /js/jquery.js Possibly upstream this?
Assignee: nobody → jason_barnabe
Status: NEW → ASSIGNED
Assignee | ||
Updated•17 years ago
|
Attachment #287315 -
Flags: review?(nelson)
Comment 2•17 years ago
|
||
hmm. TikiWiki uses MooTools instead of Jquery. Do you know how to do it using MooTools?
Assignee | ||
Comment 3•17 years ago
|
||
I'm not familiar with MooTools, but I don't see why it couldn't be done. The only thing I used JQuery for in this patch is for the cross-browser window load event. I'll revise the patch to use MooTools if you don't get to it first.
Comment 4•16 years ago
|
||
can you try revising this for MooTools? I read somewhere on the web that MooTools has js vars that clash with JQuery. Not sure if that is true but if that is the case, and since MooTools is in Tiki, I rather not take the risk.
Comment 5•16 years ago
|
||
Comment on attachment 287315 [details] [diff] [review] patch review- based on comments
Attachment #287315 -
Flags: review?(nelson) → review-
Assignee | ||
Updated•16 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•16 years ago
|
Status: RESOLVED → VERIFIED
Updated•15 years ago
|
Whiteboard: tiki_triage
Comment 7•14 years ago
|
||
heh, Tiki now uses jQuery And I think the feature was added. Let's test on http://tiki-trunk.mozilla.com/ and if it's not done, let's upstream.
Whiteboard: tiki_triage → tiki_test
You need to log in
before you can comment on or make changes to this bug.
Description
•