public inbox for [email protected]
help / color / mirror / Atom feedFrom: Dave Page <[email protected]>
To: Rohit Bhati <[email protected]>
Cc: [email protected]
Subject: Re: Require suggestion for issue #6513
Date: Wed, 27 Nov 2024 16:59:21 +0000
Message-ID: <CA+OCxozsq3n+aKTERxN3JtcowEGk2RhpHg13P3robQ_MWZVGGg@mail.gmail.com> (raw)
In-Reply-To: <CABscKVj=Ti2kiW37our1ANJSTcgU3hZU_vMUYytLuNr3hqt6Xg@mail.gmail.com>
References: <CABscKVht=_djBvUns0Pio_TDCci76UAYbPsLmYj0Ek3BEbEk+w@mail.gmail.com>
<CANVC+RETF3De3n35mK1Zi1e6tesLpYQMQ5YQmETK+W5e7kgp=w@mail.gmail.com>
<CABscKVj=Ti2kiW37our1ANJSTcgU3hZU_vMUYytLuNr3hqt6Xg@mail.gmail.com>
Hi Rohit!
On Wed, 27 Nov 2024 at 07:12, Rohit Bhati <[email protected]>
wrote:
> Hi Dave,
>
> I hope you're doing well. I would greatly appreciate it if you could
> provide your suggestions also on this issue.
>
I'm fine with the button label changes. We should make sure the text is
structured consistently though:
Are you sure you want to disconnect from the database "FooBar"?
Are you sure you want to delete the database "FooBar"?
For the delete option, I would also:
- Make Cancel the default option.
- Colour the button ("danger" in the palette)
Thanks!
> 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
>>>
>>>
>>>
>>
--
Dave Page
pgAdmin: https://www.pgadmin.org
PostgreSQL: https://www.postgresql.org
pgEdge: https://www.pgedge.com
view thread (7+ messages)
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], [email protected]
Subject: Re: Require suggestion for issue #6513
In-Reply-To: <CA+OCxozsq3n+aKTERxN3JtcowEGk2RhpHg13P3robQ_MWZVGGg@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