This needs to be reviewed and integrated.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | bluecheese-dande-kjay-cockpit.patch | 696 bytes | kjay |
| kjay-dne.patch | 15.16 KB | drumm |
This needs to be reviewed and integrated.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | bluecheese-dande-kjay-cockpit.patch | 696 bytes | kjay |
| kjay-dne.patch | 15.16 KB | drumm |
Comments
Comment #1
drummComment #2
drummI know table padding needs to be on all tables, not just releases and issues.
The issue-cockpit block should ideally get back into project_issue module itself.
I'd like feedback from project* developers and/or csevb10 on the rest.
Comment #3
drummComment #4
c4rl commentedtag
Comment #5
c4rl commentedThis is also likely to require re-roll given #893172: Refactor CSS to conform to coding standards
Comment #6
kjay commentedDoing another pass through this patch to see what is left to implement. Attached patch is the issue cockpit code redone for the current cockpit HTML. This patch tidies the form.
Also setting to normal priority as the major parts are in place I believe.
Comment #7
drummLooks good to me. Are we missing anything else?