public inbox for [email protected]  
help / color / mirror / Atom feed
[pgAdmin][RM-6178]: Cannot change background color with servers Throws js Exceptions
2+ messages / 2 participants
[nested] [flat]

* [pgAdmin][RM-6178]: Cannot change background color with servers Throws js Exceptions
@ 2021-02-05 06:26  Nikhil Mohite <[email protected]>
  0 siblings, 1 reply; 2+ messages in thread

From: Nikhil Mohite @ 2021-02-05 06:26 UTC (permalink / raw)
  To: pgadmin-hackers

Hi Team,

Please find the attached patch for RM-6178
<https://redmine.postgresql.org/issues/6178;: Cannot change background
color with servers Throws js Exceptions.
Updated the parameters for color picker functions.


-- 
*Thanks & Regards,*
*Nikhil Mohite*
*Software Engineer.*
*EDB Postgres* <https://www.enterprisedb.com/;
*Mob.No: +91-7798364578.*


Attachments:

  [application/octet-stream] RM_6178.patch (801B, 3-RM_6178.patch)
  download | inline diff:
diff --git a/web/pgadmin/static/js/backform.pgadmin.js b/web/pgadmin/static/js/backform.pgadmin.js
index 59d24ac5..85c703e2 100644
--- a/web/pgadmin/static/js/backform.pgadmin.js
+++ b/web/pgadmin/static/js/backform.pgadmin.js
@@ -3207,12 +3207,12 @@ define([
         });
       }).on('clear', (instance) => {
         this.applyColor(name, instance, null);
-      }).on('change', (color, instance) => {
+      }).on('change', (color, source, instance) => {
         this.applyColor(name, instance, color);
       }).on('show', (color, instance) => {
         const {palette} = instance.getRoot().palette;
         palette.focus();
-      }).on('hide', (color, instance) => {
+      }).on('hide', (instance) => {
         const button = instance.getRoot().button;
         button.focus();
       });


^ permalink  raw  reply  [nested|flat] 2+ messages in thread

* Re: [pgAdmin][RM-6178]: Cannot change background color with servers Throws js Exceptions
@ 2021-02-05 08:46  Akshay Joshi <[email protected]>
  parent: Nikhil Mohite <[email protected]>
  0 siblings, 0 replies; 2+ messages in thread

From: Akshay Joshi @ 2021-02-05 08:46 UTC (permalink / raw)
  To: Nikhil Mohite <[email protected]>; +Cc: pgadmin-hackers

Thanks, patch applied.

On Fri, Feb 5, 2021 at 11:56 AM Nikhil Mohite <
[email protected]> wrote:

> Hi Team,
>
> Please find the attached patch for RM-6178
> <https://redmine.postgresql.org/issues/6178;: Cannot change background
> color with servers Throws js Exceptions.
> Updated the parameters for color picker functions.
>
>
> --
> *Thanks & Regards,*
> *Nikhil Mohite*
> *Software Engineer.*
> *EDB Postgres* <https://www.enterprisedb.com/;
> *Mob.No: +91-7798364578.*
>


-- 
*Thanks & Regards*
*Akshay Joshi*
*pgAdmin Hacker | Principal Software Architect*
*EDB Postgres <http://edbpostgres.com>*

*Mobile: +91 976-788-8246*


^ permalink  raw  reply  [nested|flat] 2+ messages in thread


end of thread, other threads:[~2021-02-05 08:46 UTC | newest]

Thread overview: 2+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2021-02-05 06:26 [pgAdmin][RM-6178]: Cannot change background color with servers Throws js Exceptions Nikhil Mohite <[email protected]>
2021-02-05 08:46 ` Akshay Joshi <[email protected]>

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox