Opened 4 years ago

Closed 4 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@…


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.


Attachments (1)

column_hooks_and_unlocked_status.patch (7.6 KB) - added by ziba 4 years ago.

Download all attachments as: .zip

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 4 years ago by alec

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.

comment:3 Changed 4 years ago by alec

Please, review and update patch for current trunk.

comment:4 Changed 4 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: See TracTickets for help on using tickets.