Re: Include patch id in commitfest page
От | Peter Eisentraut |
---|---|
Тема | Re: Include patch id in commitfest page |
Дата | |
Msg-id | ff9d7921-f4e3-498a-9dba-ddd28fdb359f@eisentraut.org обсуждение исходный текст |
Ответы |
Re: Include patch id in commitfest page
Re: Include patch id in commitfest page |
Список | pgsql-hackers |
I want to register that I'm not a fan of this change: https://git.postgresql.org/gitweb/?p=pgcommitfest2.git;a=commit;h=eee60a5332ac0745071086477ff4e189b080d3ab 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.
В списке pgsql-hackers по дате отправления: