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