public inbox for [email protected]  

help / color / mirror / Atom feed

From: Rohit Bhati <[email protected]>
To: [email protected]
Subject: Re: Require suggestion for issue #6513
Date: Wed, 27 Nov 2024 12:41:59 +0530
Message-ID: <CABscKVj=Ti2kiW37our1ANJSTcgU3hZU_vMUYytLuNr3hqt6Xg@mail.gmail.com> (raw)
In-Reply-To: <CANVC+RETF3De3n35mK1Zi1e6tesLpYQMQ5YQmETK+W5e7kgp=w@mail.gmail.com>
References: <CABscKVht=_djBvUns0Pio_TDCci76UAYbPsLmYj0Ek3BEbEk+w@mail.gmail.com>
	<CANVC+RETF3De3n35mK1Zi1e6tesLpYQMQ5YQmETK+W5e7kgp=w@mail.gmail.com>

Hi Dave,

I hope you're doing well. I would greatly appreciate it if you could
provide your suggestions also on this issue.

Thank you in advance.

Best regards,
Rohit Bhati


On Thu, Nov 21, 2024 at 11:40 PM Usman Khan <[email protected]> wrote:

> Yes, I think it would be nice to do it across all appropriate places.
> 'Disconnect' is available for Server and Database, while Delete is
> available across all database objects in the tree area.
>
>
>
>
>
> On Thu, Nov 21, 2024 at 12:48 PM Rohit Bhati <[email protected]>
> wrote:
>
>> Hi Hackers,
>>
>> I have been working on issue #6513
>> <https://github.com/pgadmin-org/pgadmin4/issues/6513; and have
>> implemented some changes based on user suggestions. Here’s a summary of the
>> modifications I made locally:
>>
>>    1. *Button Label Changes*:
>>       - Updated the labels of the buttons to "Delete" or "Disconnect"
>>       for affirmative actions.
>>       - Changed the label "No" to "Cancel" for negative actions.
>>    2. *Icon Additions*:
>>       - Added appropriate icons for the "Disconnect" and "Delete"
>>       actions.
>>
>> I would like your input on whether these changes should be applied
>> universally across all delete and disconnect scenarios, or if they should
>> be limited to just the database delete and disconnect actions.
>>
>> Please find the attached screenshots showcasing the changes.
>>
>>
>> Thank you for your feedback!
>>
>> Best regards,
>> Rohit Bhati
>>
>>
>>
>

view thread (7+ 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: Require suggestion for issue #6513
  In-Reply-To: <CABscKVj=Ti2kiW37our1ANJSTcgU3hZU_vMUYytLuNr3hqt6Xg@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