Opened 4 years ago
Closed 3 years ago
#1486257 closed Feature Patches (fixed)
Server-side column flexibility patch
| Reported by: | ziba | Owned by: | |
|---|---|---|---|
| Priority: | 5 | Milestone: | 0.4-beta |
| Component: | Core functionality | Version: | 0.3-stable |
| Severity: | normal | Keywords: | columns, re-ordering |
| Cc: | technetux@…, steinhof@… |
Description
Greetings! I think there was a bit of a misunderstanding about "Reorderable Columns" #1485806
This patch is an updated version of the one from #1485806. The original ticket was marked as duplicate of #1485795. This was understandable because the purpose of the patch was not optimally articulated. However, it is not a duplicate. #1485806 is an implementation of a feature that lays the groundwork for the more advanced request made in #1485795.
This patch increases column flexibility in two ways:
#1. It introduces two new plugin hooks. One for inserting column headers and one for inserting column body cells. We've used this hook to implement a checkboxes column plugin, for example.
#2. It removes the assumptions that the status column is on the far left. This means that the keyword "status" can be used in the config variable $rcmail_configlist_cols? to place the column in the desired order. With this improvement, any future implementation of client-side column re-ordering would be able to move the status column.
Thanks!
Attachments (1)
Change History (5)
Changed 4 years ago by ziba
comment:1 Changed 4 years ago by alec
- Milestone changed from later to 0.4-beta
comment:2 Changed 3 years ago by alec
comment:3 Changed 3 years ago by alec
Please, review and update patch for current trunk.
comment:4 Changed 3 years ago by alec
- Resolution set to fixed
- Status changed from new to closed
Please re-open if we need something from this patch.
Note: 'status' column is currently removed in devel-threads branch, and will be probably removed also in trunk. Also some code from patch is there. So, we should review this after merge.