- Notifications
You must be signed in to change notification settings - Fork88
templates: add optional Series ID buttons to the patch list#571
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
rossburton commentedNov 22, 2023
rossburton commentedNov 22, 2023
I'm torn on whether the column should just be called "ID" and let its position clarifywhat ID it refers to. |
rossburton commentedNov 22, 2023
Maybe the ID column should just be removed, and the button inline with the patch or series name? |
stephenfin commentedJan 23, 2024
I wonder if, rather than adding another column, we could add the |
If the user has turned on display of patch IDs, add clickable series IDbuttons inline with the series name.Signed-off-by: Ross Burton <ross.burton@arm.com>
rossburton commentedJan 25, 2024
rossburton commentedJan 25, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
The good thing about two columns is it forces the alignment... |
rossburton commentedFeb 3, 2024
Polite ping |


Uh oh!
There was an error while loading.Please reload this page.
If the user has turned on display of patch IDs, add another column with the series IDs to the patch list template.
Note that I've not tested this in production because I've not yet setup a local instance...