I find myself wanting to know the serial number of my certs much more often than the issuance date. We should still allow the user to see the Issued On column, but it should be off by default. The Serial Number column should be on by default.
If the serial number needs to be truncated, the left side should be truncated. For example: ...41:74:AA:21
This patch provides a partial fix to the issue. This is what it does : puts the serial number as a new col. Issue Date column still exists ... but hidden by default. Note: 'hidden' property of columns being persistent, 'issue date' would be hidden only on new profiles. On old profiles both 'serial number' and 'issue date' would show up. What it does not : The serial number col is aligned to the left - which means it get truncated as 41:74:AA:21.... rather than ....41:74:AA:21. Shall try to resolve this - at present it looks like align does not work ok with Outliners and enforces all columns to get left aligned.
Comment on attachment 51300 [details] [diff] [review] patch why are we actually keeping the issued on column? Do we actually want to remove it? If so, then we should actually remove it and remove the associated code.
As I understand, we still want to have the 'issued on' column - but turned off by default, unless the user explicitly turns it on ... the serial number column would be turned on by default...
Comment on attachment 51300 [details] [diff] [review] patch in that case, r=javi
Comment on attachment 51300 [details] [diff] [review] patch do you really need the align="end" on the outlinercol?
Not really - I was trying to get the outliner col left aligned, but doesn't seem to work, even with align=left - any suggestions on that ..? Here's the new patch anyway .. removed the align property.
Comment on attachment 56190 [details] [diff] [review] new patch sr=hewitt
Patch checked in... But the fix is partial, in the sense that the serial number are still left aligned that is, tunncated as 41:74....
clearing patch keyword since patch is in
Changing summary, since one of the bugs is fixed.
reassign former PSM engineers' bugs to nobody