public inbox for [email protected]
help / color / mirror / Atom feedFrom: Aditya Toshniwal <[email protected]>
To: pgadmin-hackers <[email protected]>
Subject: [pgAdmin][RM3538] Reset button does not get enabled till all the mandatory fields provided with value in any dialogue.
Date: Fri, 15 Nov 2019 10:59:38 +0530
Message-ID: <CAM9w-_=KRPbFbc=HmpN_LROh8eTf9tvwvFOydfB4+_YN+vNXzw@mail.gmail.com> (raw)
Hi Hackers,
Attached is the patch to enable reset button of dialog if any field is
changed. Currently, it is enabled only if all the mandatory fields are
provided.
--
Thanks and Regards,
Aditya Toshniwal
Sr. Software Engineer | EnterpriseDB India | Pune
"Don't Complain about Heat, Plant a TREE"
Attachments:
[application/octet-stream] RM3538.patch (650B, 3-RM3538.patch)
download | inline diff:
diff --git a/web/pgadmin/browser/static/js/node.js b/web/pgadmin/browser/static/js/node.js
index 0f95b1b2d..90e4fb611 100644
--- a/web/pgadmin/browser/static/js/node.js
+++ b/web/pgadmin/browser/static/js/node.js
@@ -346,7 +346,12 @@ define('pgadmin.browser.node', [
this.empty().css('visibility', 'hidden');
}.bind(that.statusBar));
}
- callback(true);
+
+ var sessHasChanged = false;
+ if(this.sessChanged && this.sessChanged()){
+ sessHasChanged = true;
+ }
+ callback(true, sessHasChanged);
return true;
};
view thread (2+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected]
Subject: Re: [pgAdmin][RM3538] Reset button does not get enabled till all the mandatory fields provided with value in any dialogue.
In-Reply-To: <CAM9w-_=KRPbFbc=HmpN_LROh8eTf9tvwvFOydfB4+_YN+vNXzw@mail.gmail.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox