Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Server-side column flexibility patch #2497

Closed
rcubetrac opened this issue Oct 30, 2009 · 5 comments
Closed

Server-side column flexibility patch #2497

rcubetrac opened this issue Oct 30, 2009 · 5 comments

Comments

@rcubetrac
Copy link

Reported by ziba on 30 Oct 2009 17:36 UTC as Trac ticket #1486257

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_config['list_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!

Keywords: columns, re-ordering
Migrated-From: http://trac.roundcube.net/ticket/1486257

@rcubetrac
Copy link
Author

Milestone changed by @alecpl on 2 Nov 2009 07:26 UTC

later => 0.4-beta

@rcubetrac
Copy link
Author

Comment by @alecpl on 26 Nov 2009 17:48 UTC

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.

@rcubetrac
Copy link
Author

Comment by @alecpl on 19 Mar 2010 20:22 UTC

Please, review and update patch for current trunk.

@rcubetrac
Copy link
Author

Comment by @alecpl on 20 Apr 2010 12:31 UTC

Please re-open if we need something from this patch.

@rcubetrac
Copy link
Author

Status changed by @alecpl on 20 Apr 2010 12:31 UTC

new => closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant