Re: Require suggestion for issue #6513

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 <umk555(at)gmail(dot)com> 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 <rohit(dot)bhati(at)enterprisedb(dot)com>
> 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
>>
>>
>>
>