acm-uiuc / liquid

ACM@UIUC's website code (Django)
acm.uiuc.edu
15 stars 33 forks source link

Sortable Tables #39

Open nhandler opened 12 years ago

nhandler commented 12 years ago

Many of the sections under the Intranet display their results in tables. These tables can be quite large. It would be handy if you could click on column headings to quickly adjust how the table is sorted.

reedlabotz commented 12 years ago

Yeah, we can do that no problem.

On Wed, Aug 15, 2012 at 6:41 PM, Nathan Handler notifications@github.comwrote:

Many of the sections under the Intranet display their results in tables. These tables can be quite large. It would be handy if you could click on column headings to quickly adjust how the table is sorted.

— Reply to this email directly or view it on GitHubhttps://github.com/acm-uiuc/liquid/issues/39.

colegleason commented 12 years ago

Rebecca would like this for resume book as well.

reedlabotz commented 12 years ago

I like that Rebecca's name now appears multiple times in the liquid github

chorn21 commented 12 years ago

Who's Rebecca?

On Thu, Sep 27, 2012 at 6:09 PM, Reed La Botz notifications@github.comwrote:

I like that Rebecca's name now appears multiple times in the liquid github

— Reply to this email directly or view it on GitHubhttps://github.com/acm-uiuc/liquid/issues/39#issuecomment-8958178.

Caroline Horn University of Illinois at Urbana-Champaign - Undergraduate College of Liberal Arts & Sciences - Mathematics & Computer Science chorn21@gmail.com

dylnuge commented 12 years ago

Facebook recruiter.

chorn21 commented 12 years ago

Sent from my iPhone

On Sep 27, 2012, at 11:54 PM, Dylan Nugent notifications@github.com wrote:

Facebook recruiter.

— Reply to this email directly or view it on GitHub.

dylnuge commented 12 years ago

Sorry, hard to tell over the internet. I make sure people have their questions answered, even if they already know the answer. Better than leaving someone in the dark :)

ace-n commented 11 years ago

Is this something we are still interested in doing?

if (yes) { carry_on(); } else { this.getIssue().close(); }

calvin-shirley commented 11 years ago

It would be nice, yes.

ace-n commented 10 years ago

My pull request implementing this for the resume book was merged.

Are there any tables that need to be sort-ified (the most obvious candidate being the member database), or are we OK to close this issue?

colegleason commented 10 years ago

I think we want pretty much all tables to have this behavior, but that suggests we should just get some javascript library to handle it.

ace-n commented 10 years ago

^ Like I said in my previous comment, clientside pagination is my main contention point. AFAIK it is possible but would require massive data downloading.

I tried to find a more general Django library to take care of this, but I couldn't find anything that was definitively what we wanted. (I don't know how to write my own, though I can certainly take a look into it).

EDIT: I may be able to make a sortify() function that makes a table sortable in the view. Without writing a library, I don't know how to make it concise.

colegleason commented 10 years ago

I'm not sure it's significant enough to slow down the end-user. Maybe I'm wrong though.

brendanjryan commented 10 years ago

@ace-n what is preventing you from loading the first page of results and then lazy-loading in the rest?

ace-n commented 10 years ago

That could work. But if you are going to sort things clientside, you either need a really clever algorithm (which I won't rule out) or all the data to sort. (Unless you are just going to AJAX sorted data from the server.)

On Thu, Jan 23, 2014 at 11:20 PM, Brendan Ryan notifications@github.comwrote:

@ace-n https://github.com/ace-n what is preventing you from loading the first page of results and then lazy-loading in the rest?

— Reply to this email directly or view it on GitHubhttps://github.com/acm-uiuc/liquid/issues/39#issuecomment-33198245 .

colegleason commented 10 years ago

Look, how much info is in our largest table? A few KB?

ace-n commented 10 years ago

I'm just concerned about scalabilty. =P

On Thu, Jan 23, 2014 at 11:34 PM, Cole Gleason notifications@github.comwrote:

Look, how much info is in our largest table? A few KB?

— Reply to this email directly or view it on GitHubhttps://github.com/acm-uiuc/liquid/issues/39#issuecomment-33198690 .