public inbox for [email protected]
help / color / mirror / Atom feedFrom: Peter Eisentraut <[email protected]>
To: Jelte Fennema-Nio <[email protected]>
To: Magnus Hagander <[email protected]>
Cc: pgsql-hackers <[email protected]>
Subject: Re: Include patch id in commitfest page
Date: Sat, 11 Jan 2025 10:40:26 +0100
Message-ID: <[email protected]> (raw)
I want to register that I'm not a fan of this change:
https://git.postgresql.org/gitweb/?p=pgcommitfest2.git;a=commit;h=eee60a5332ac0745071086477ff4e189b0...
Rationale: It puts the least important information first. For the most
part, the CF entry number is irrelevant, so it shouldn't be shown in
such a prominent position.
Maybe
<a>title here</a> (number here)
would be a compromise. But looking at the original argument:
"""
Clicking the patch header would sort by the patch ID, but that ID would
not actually be visible. Effectively resulting in random ordering for
the user, so let's just show the ID.
"""
I would suggest that the actual fix would be to order by title instead
of ID.
Or just turn the sorting off.
view thread (2+ 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], [email protected]
Subject: Re: Include patch id in commitfest page
In-Reply-To: <[email protected]>
* 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