Patch #32014
closed
Rounded corners on table.list elements
Added by Anonymous over 6 years ago. Updated over 6 years ago.
Description
Same motivation as in #32013
The patch doesn't make calendar and gantt tables rounded though, due to more complex structure that they have, hence requiring different approach.
Files
| rounded-tables(excl-calendar+gantt+mypage).patch (846 Bytes) rounded-tables(excl-calendar+gantt+mypage).patch | Anonymous, 2019-09-04 18:55 | ||
| roundedlisttables.png (40.5 KB) roundedlisttables.png | Anonymous, 2019-09-04 19:31 | ||
| roundedlisttables2.png (27.1 KB) roundedlisttables2.png | Anonymous, 2019-09-04 19:31 | ||
| rounded-tables(excl-calendar+gantt+mypage)-v2.patch (859 Bytes) rounded-tables(excl-calendar+gantt+mypage)-v2.patch | Anonymous, 2019-09-27 20:43 |
Related issues
Updated by Anonymous over 6 years ago
- File roundedlisttables.png roundedlisttables.png added
- File roundedlisttables2.png roundedlisttables2.png added
Updated by Bernhard Rohloff over 6 years ago
- Related to Patch #32013: Rounded corners of the main menu added
Updated by Go MAEDA over 6 years ago
The patch removes border-collapse property. Do you think it is OK? (sorry if this is a stupid question. I am not a CSS expert).
Updated by Anonymous over 6 years ago
Go, roundness with border-radius on table elements can't seem to be achieved if border-collapse: collapse; is present, so I cleared it in a favor of border-spacing: 0; for this reason, which it did seem like does the same thing, only, unlike border-collapse, it doesn't block border-radius to be applied.
Updated by Bernhard Rohloff over 6 years ago
Yes, as Antonio wrote, it's necessary to get the border radius and border-spacing: 0; compensates it.
One notice of caution: I played with the border radius and found out that the cells don't get clipped to the round shape. Maybe an additional overflow: hidden; is missing here.
Updated by Anonymous over 6 years ago
- File rounded-tables(excl-calendar+gantt+mypage)-v2.patch rounded-tables(excl-calendar+gantt+mypage)-v2.patch added
Bernhard Rohloff wrote:
Maybe an additional
overflow: hidden;is missing here.
Not that it's a biggie, as our border radius doesn't seem to be that big anyway, for cells under to stick out, it's only 3px after all, but here is the patch with hidden overflow just in case ;-)
Updated by Go MAEDA over 6 years ago
- Subject changed from Rounded corners on table/list elements to Rounded corners on table.list elements
- Status changed from New to Closed
- Assignee set to Go MAEDA
- Target version set to 4.1.0
Committed the patch. Thank you for improving Redmine.
Updated by Anonymous over 6 years ago
Go MAEDA wrote:
I will revert this change because the change caused other changes such as #32165 and #32166 and I cannot predict how much the impact will spread further.
#32165 and #32166 would be the last patches for the tables anyway, and it's really to be over with now after those two really small patches :D
Just calendar and gantt were styled and structured differently than list.tables and also each-other, hence why required separate patches.
Either way, I think it's better to keep it like that rather than reverting it, since gantt and calendar seem like the only exceptions anyway. Then deliver those two in later versions (if release will happen before they ready) if necessary.
We should probably add them as related too.
Updated by Go MAEDA over 6 years ago
- Related to Patch #32165: Rounded corners on table.cal added
Updated by Go MAEDA over 6 years ago
- Related to Patch #32166: Rounded corners on table.gantt-table added