Re: Copy script button cleanup

From: Magnus Hagander <magnus(at)hagander(dot)net>
To: Dave Page <dpage(at)pgadmin(dot)org>
Cc: PostgreSQL WWW <pgsql-www(at)lists(dot)postgresql(dot)org>
Subject: Re: Copy script button cleanup
Date: 2025-12-02 14:41:07
Message-ID: CABUevExmHVYwddKvFUWkjY8Ev-xAE2p0-mP0wJqJDs1bazMn7A@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-www

On Tue, 2 Dec 2025 at 12:30, Dave Page <dpage(at)pgadmin(dot)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, and I'm told you're not supposed to use "var" for variables in js due
to the broken scoping :)

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...

--
Magnus Hagander
Me: https://www.hagander.net/ <http://www.hagander.net/>
Work: https://www.redpill-linpro.com/ <http://www.redpill-linpro.com/>

In response to

Responses

Browse pgsql-www by date

  From Date Subject
Next Message Magnus Hagander 2025-12-02 21:30:44 Re: Images in news mails
Previous Message Dave Page 2025-12-02 11:29:40 Copy script button cleanup