Re: [PATCH] Add CSS to support discoverable ids in the public documentation website - Mailing list pgsql-www

From Brar Piening
Subject Re: [PATCH] Add CSS to support discoverable ids in the public documentation website
Date
Msg-id ee778f0b-e59b-1bde-87a5-a2d1316d49e3@gmx.de
Whole thread Raw
In response to Re: [PATCH] Add CSS to support discoverable ids in the public documentation website  (Melanie Plageman <melanieplageman@gmail.com>)
Responses Re: [PATCH] Add CSS to support discoverable ids in the public documentation website
List pgsql-www
On 19.04.2023 at 22:21, Melanie Plageman wrote:
> On Wed, Apr 19, 2023 at 4:08 PM Jonathan S. Katz <jkatz@postgresql.org> wrote:
>> On 4/13/23 9:41 AM, Brar Piening wrote:
>>> Attached is the (unmodified) patch I initially sent to pgsql-hackers in
>>> [2]. Please see the thread in pgsql-hackers for the discussion so far.
>> I personally like the paperclip given how it's synonymous with "link",
>> but that may also give the illusion that we also automatically copy the
>> link to the clipboard. Perhaps "#" is good enough?
> I find the "#" pretty meaningless and the paper clip a bit silly.
> Personally, I think the link icon is the nicest. David pointed out [3]
> does this, and it is quite visually appealing.


I have no qualifications or whatsoever to comment on the visual design,
so I will not be involved in any bike shedding but please note that the
patch here does not and cannot chose the symbol we are displaying. Being
pure CSS it only changes the default color and visibility of the symbol.
The symbol was part of the xslt modifications in the patch that I sent
to hackers and was commited in commit e2922702a3. If you really think
that changing the symbol is important or useful, you'll probably have to
carry the discussion back to hackers (maybe to the original thread).

Regards,
Brar




pgsql-www by date:

Previous
From: Melanie Plageman
Date:
Subject: Re: [PATCH] Add CSS to support discoverable ids in the public documentation website
Next
From: Stefanie Janine Stölting
Date:
Subject: Patch to clean up and add missing GSoC pages