Issue #129: Visually distinguish meetings in list view - fedocal

fedocal

#129 Visually distinguish meetings in list view

Closed: Fixed None Opened 9 years ago by lbrabec.

Current design of list view is somehow confusing - at first sight, one can't tell where meeting description starts and where it ends. Some kind of separator between meetings would be nice.

Also, since meeting detail is now pretty polished and there is 'blue box', it could be more readable if just part of meeting description was shown and rest of it would be accessible through click.

I did quickfix for this, see attachment for patch.

pingou commented 9 years ago

This patch fails to apply for me on both the master and the tests branch

{{{
Applying: listview quickfix
/home/pingou/repos/gitrepo/fedocal/.git/rebase-apply/patch:79: trailing whitespace.
{% autoescape off%} {{ meeting.meeting_information }}
/home/pingou/repos/gitrepo/fedocal/.git/rebase-apply/patch:81: trailing whitespace.
href="{{ url_for('view_meeting',
error: patch failed: fedocal/init.py:439
error: fedocal/init.py: patch does not apply
Patch failed at 0001 listview quickfix
The copy of the patch that failed is found in:
/home/pingou/repos/gitrepo/fedocal/.git/rebase-apply/patch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
}}}

pingou commented 9 years ago

Also, for this part:
{{{
+ for i in range(len(meetings)):
+ lead_par = meetings[i].meeting_information[:100]
+ meetings[i].meeting_information = lead_par + "..."
}}}

I think we should be able to do this directly in the template rather than iterating through all the meetings in the controller

lbrabec commented 9 years ago

new patch (againts fedorahosted repo)
fedocal-list-enhance.patch

lbrabec commented 9 years ago

Ok, again, diff against fedorahosted repo. Tried, works for me.

lbrabec commented 9 years ago

Yes, the lead paragraph part should be in model. However this was just preview meant only for showing what I had in mind not a patch for production.

Also, first X characters of description can cut text in the middle of word. Is it problem? Nicer option could be first several words or something like that.

pingou commented 9 years ago

One option might be to create a new jinja filter and maybe https://docs.python.org/2/library/textwrap.html can help getting the first X characters correctly

lbrabec commented 9 years ago

I like the jinja filter option.

Textwrap module looks pretty good, something like this:
{{{
'\n'.join(textwrap.wrap(a)[:3])+"..."
}}}
could do the work (returns first 3 lines).

What about 'show more' link after the lead paragraph, is it needed?

pingou commented 9 years ago

I was wondering about it, I guess it's not really since the regular link is just in the next cell

lbrabec commented 9 years ago

Yep, that's my argument against it too, but I'm not sure if it is clear enough that the 'blue box' is clickable link. I'll ask around what others think.

pingou commented 9 years ago

It's the same box as in the calendar view, but sure let's check if it is clear enough

lbrabec commented 9 years ago

The feedback is that 'show more' or something like little arrow or a pictogram (like wiki uses for external links) at the end of lead paragraph would be better than only clickable 'blue box'.

Also kparal suggested that about 8 lines of text would be best (enough info from it).

pingou commented 9 years ago

8 lines seems a bit much to me but ok, we can always tweak that later.

Let's go for a small arrow/pictogram (with the title show more) then :)