Currently, we have views for the source tab. We can edit them.

For the coded lists, we have fixed pagination limits. And no way to reconfigure them.
Combined with the issue that we lack capabilities to persist selections through multiple pages, this is a severe UX issue.

For us, the most simple solution is to disable pagination and select in one single huge list.
Not my favorite situation. But we can live with it and it's a good start.

Something like this would fix our situation:
# drush vset tmgmt_page_limit 1000

Comments

berdir’s picture

Note that large lists are quite a performance hog, it's often not just a single query but we need to load additional data or we load whole entities. Loading a 1000 nodes including translation medata is going to be quite a hit.

miro_dietiker’s picture

Perfectly agree. That's why i'm not asking for such a default.

blueminds’s picture

Status: Active » Needs review
StatusFileSize
new2.17 KB

Something like this?

berdir’s picture

yes, but I think this should be the default for those functions instead of specifying it explicitly when we use it.

I even hat the crazy idea what we could try to place the variable_get() in exported default views, might work but there's a relatively high chance of overriding it again when we change them.

berdir’s picture

Status: Needs review » Needs work
miro_dietiker’s picture

You could write a test for it -- so we won't forget it. ;-)

blueminds’s picture

Status: Needs work » Needs review
StatusFileSize
new5.34 KB
new7.32 KB

here we go

miro_dietiker’s picture

Status: Needs review » Fixed

Wonderful. Fixed it.

There's some strange debug output from Views though we should look into...
BTW: I wonder why we had less tests in january 2014 than in december 2013..?!

berdir’s picture

Title: Define pagination limit with variable » (test problem follow-up) Define pagination limit with variable
Status: Fixed » Active

As discussed, re-open so that we don't forget to look into that debug thing.

berdir’s picture

Status: Active » Needs review
StatusFileSize
new1.06 KB

Found it.

miro_dietiker’s picture

Status: Needs review » Fixed

Added "translatable" to the comment and committed it.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.