individual patch detail page design changes
raxel at google.com
Thu Jul 15 21:47:40 AEST 2021
Forgot to mention, the list action bar isn't exporting correctly from
Figma. The screenshot of the implementation I shared in
what it should look like.
On Thu, Jul 15, 2021 at 7:45 AM Raxel Gutierrez <raxel at google.com> wrote:
> I attached a long image of a patch detail page mockup I made. I
> recommend zooming in and scrolling down to see any details, but I'll
> be mentioning the important changes. First, an idea from Emily about
> having a table that connects an individual patch with its various
> revisions. The table also adds a 'Comments' column that includes the
> unaddressed and addressed comment right after icons that help to
> indicate the status. The tags column is carried over from the regular
> patch list. Also, the 'Comments' column is something that I think
> should be added to the central patch list table.
> Some changes that were inspired by FDO's version of Patchwork is
> having the 'Patch' section of a patch be right after the 'Commit
> Message', making it easier to find rather than after what can be many
> comments for a patch. Additionally, there is now a 'Hide' button to be
> able to collapse the 'Patch' section if desired. Also, some
> controls/info has been right-aligned for easier parsing of the screen.
> Last but not least, each comment header includes a label to indicate
> whether a comment is addressed or unaddressed as well as buttons to
> change that state of the comment. The buttons will be the intended
> users of patch authors, delegates, and maintainers. The visibility of
> the controls is up in the air for the un-authorized users, but having
> it hidden would be the simplest way to implement first. The
> labels/buttons are consistent with the proposed 'Comments' column for
> patch lists. Also, this added feature should work well and be
> transferred over to the TODO list currently available in Patchwork.
More information about the Patchwork