Re: Copy script button cleanup - Mailing list pgsql-www

From Magnus Hagander
Subject Re: Copy script button cleanup
Date
Msg-id CABUevEw4DSsQemiDxn2WQffp2J+1Q7iEBuBTx_zNwf5tqgg6Rg@mail.gmail.com
Whole thread Raw
In response to Re: Copy script button cleanup  (Dave Page <dpage@pgadmin.org>)
Responses Re: Copy script button cleanup
List pgsql-www
On Wed, 3 Dec 2025 at 11:38, Dave Page <dpage@pgadmin.org> wrote:


On Tue, 2 Dec 2025 at 14:41, Magnus Hagander <magnus@hagander.net> wrote:


On Tue, 2 Dec 2025 at 12:30, Dave Page <dpage@pgadmin.org> wrote:
The Copy Script buttons on the Linux package installation pages look horrific. See before.png.

The attached patch makes them look much nicer, with proper sizing and an icon based button. On click, it briefly changes to a green checkmark to show confirmation. See the after screenshots.

I'll apply this in a couple of days if there are no objections.


LGTM in general and certainly is a lot prettier.

Can you explain what that "fallback for text based buttons" is?  Aren't they all replaced with icons?

Oh, yeah. That's no longer needed now the patch is complete. I'll remove it.
 

Oh, and I'm told you're not supposed to use "var" for variables in js due to the broken scoping :)

Old habits die hard :-(
 

And while you're poking at it, one thing I've really wanted is a "copy without the sudo parts". But I'm guessing that would be a lot more complicated than what you're fixing here...

Ask and thou shalt receive.

Sweet! :)


2 patches attached. Both fix the original issue. One also adds the no sudo button.


Looks good in a (fairly brief) check. Thanks!
 
--

pgsql-www by date:

Previous
From: Daniel Gustafsson
Date:
Subject: Re: Copy script button cleanup
Next
From: "Jonathan Gonzalez V."
Date:
Subject: Re: Support for listing the beta versions