| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0031944 | mantisbt | ui | public | 2023-01-26 13:33 | 2023-01-28 09:52 |
| Reporter | M13 | Assigned To | atrol | ||
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | resolved | Resolution | fixed | ||
| Product Version | 2.25.5 | ||||
| Target Version | 2.26.0 | Fixed in Version | 2.26.0 | ||
| Summary | 0031944: "pinning" an issue calls for not CSS code in view_all_inc.php | ||||
| Description | view_all_inc.php, line 196: <td colspan="<?php echo count( $g_columns ); ?>" bgcolor="#d3d3d3"></td> so it would always render a gray line under pinned without regard to any skins\plugins\custom css | ||||
| Steps To Reproduce | pin any issue | ||||
| Tags | No tags attached. | ||||
 |
That's exactly what is not wanted. Furthermore inline styles are bad practice due to various reasons, e.g. security see 0021908, https://stackoverflow.com/questions/2612483/whats-so-bad-about-in-line-css ... |
 |
Yes, I know, that is wahy I report it as a bug: the view_all_inc.php I'm refering to is not modified, from installation package - you have that inline there and it needs to be fixed |
|
|
Sorry @M13 I was confused by your wording and the fact that this is already fixed in latest code, https://github.com/mantisbt/mantisbt/commit/3cc574bac2a73e4761b1206c265a413d57a5e490 |
|
|
perfect, glad to hear |
